wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 886848 - Remove _skipEditorBreakpointCallbacks since it's redundant after using promises when handling breakpoints, r=past

Browse Source
This commit is contained in:
Victor Porof 2013-09-13 16:23:12 +03:00
parent aee4f1b7d2
commit 6ea2865fa2
1 changed files with 0 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
browser/devtools/debugger/debugger-controller.js
View File

@ -1212,20 +1212,6 @@ Breakpoints.prototype = {
* A map of breakpoint promises as tracked by the debugger frontend.
* The keys consist of a string representation of the breakpoint location.
*/
/**
* Skip editor breakpoint change events.
*
* This property tells the source editor event handler to skip handling of
* the BREAKPOINT_CHANGE events. This is used when the debugger adds/removes
* breakpoints from the editor. Typically, the BREAKPOINT_CHANGE event handler
* adds/removes events from the debugger, but when breakpoints are added from
* the public debugger API, we need to do things in reverse.
*
* This implementation relies on the fact that the source editor fires the
* BREAKPOINT_CHANGE events synchronously.
*/
_skipEditorBreakpointCallbacks: false,
_added: new Map(),
_removing: new Map(),
@ -1264,13 +1250,8 @@ Breakpoints.prototype = {
* The SourceEditor.EVENTS.BREAKPOINT_CHANGE event object.
*/
_onEditorBreakpointChange: function(aEvent) {
if (this._skipEditorBreakpointCallbacks) {
return;
}
this._skipEditorBreakpointCallbacks = true;
aEvent.added.forEach(this._onEditorBreakpointAdd, this);
aEvent.removed.forEach(this._onEditorBreakpointRemove, this);
this._skipEditorBreakpointCallbacks = false;
},
/**
@ -1545,9 +1526,7 @@ Breakpoints.prototype = {
// Update the editor if required.
if (!aFlags.noEditorUpdate) {
if (location.url == currentSourceUrl) {
this._skipEditorBreakpointCallbacks = true;
DebuggerView.editor.addBreakpoint(location.line - 1);
this._skipEditorBreakpointCallbacks = false;
}
}
@ -1581,9 +1560,7 @@ Breakpoints.prototype = {
// Update the editor if required.
if (!aFlags.noEditorUpdate) {
if (aLocation.url == currentSourceUrl) {
this._skipEditorBreakpointCallbacks = true;
DebuggerView.editor.removeBreakpoint(aLocation.line - 1);
this._skipEditorBreakpointCallbacks = false;
}
}
// Update the breakpoints pane if required.