Title: [172217] trunk/Source/WebInspectorUI
Revision
172217
Author
[email protected]
Date
2014-08-07 10:57:51 -0700 (Thu, 07 Aug 2014)

Log Message

Web Inspector: DebuggerManager sends spurious setBreakpointsActive commands when setting a breakpoint
https://bugs.webkit.org/show_bug.cgi?id=135674

Reviewed by David Kilzer.

* UserInterface/Controllers/DebuggerManager.js:
(WebInspector.DebuggerManager.prototype.set breakpointsEnabled): Fix a typo.

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (172216 => 172217)


--- trunk/Source/WebInspectorUI/ChangeLog	2014-08-07 17:29:52 UTC (rev 172216)
+++ trunk/Source/WebInspectorUI/ChangeLog	2014-08-07 17:57:51 UTC (rev 172217)
@@ -1,3 +1,13 @@
+2014-08-07  Brian J. Burg  <[email protected]>
+
+        Web Inspector: DebuggerManager sends spurious setBreakpointsActive commands when setting a breakpoint
+        https://bugs.webkit.org/show_bug.cgi?id=135674
+
+        Reviewed by David Kilzer.
+
+        * UserInterface/Controllers/DebuggerManager.js:
+        (WebInspector.DebuggerManager.prototype.set breakpointsEnabled): Fix a typo.
+
 2014-08-06  Brian J. Burg  <[email protected]>
 
         Web Inspector: breakpoint resolved state should not depend on all breakpoints being enabled

Modified: trunk/Source/WebInspectorUI/UserInterface/Controllers/DebuggerManager.js (172216 => 172217)


--- trunk/Source/WebInspectorUI/UserInterface/Controllers/DebuggerManager.js	2014-08-07 17:29:52 UTC (rev 172216)
+++ trunk/Source/WebInspectorUI/UserInterface/Controllers/DebuggerManager.js	2014-08-07 17:57:51 UTC (rev 172217)
@@ -101,7 +101,7 @@
 
     set breakpointsEnabled(enabled)
     {
-        if (this._breakpointsEnabled === enabled)
+        if (this._breakpointsEnabledSetting.value === enabled)
             return;
 
         this._breakpointsEnabledSetting.value = enabled;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to