Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse code

Issue 5316: Break on HTML mutation works only if the HTML panel is se…

…lected
  • Loading branch information...
commit 27b2d859da1f2513951a491f29aeb9c3c0eb216c 1 parent 924a2d9
Jan Odvarko authored March 12, 2012
5  extension/content/firebug/html/htmlPanel.js
@@ -1160,7 +1160,10 @@ Firebug.HTMLPanel.prototype = Obj.extend(WalkingPanel,
1160 1160
 
1161 1161
         Events.removeEventListener(this.panelNode.ownerDocument, "keypress", this.onKeyPress, true);
1162 1162
 
1163  
-        if (this.context.attachedMutation)
  1163
+        // Optimalization: if the HTML panel isn't currently selected remove
  1164
+        // mutation listeners. Don't do it if there are mutation breakpoints
  1165
+        // these should work even if the HTML panel is not selected (issue 5316)
  1166
+        if (this.context.attachedMutation && this.context.mutationBreakpoints.isEmpty())
1164 1167
         {
1165 1168
             this.context.attachedMutation = false;
1166 1169
 

0 notes on commit 27b2d85

Please sign in to comment.
Something went wrong with that request. Please try again.