2010-12-30 Pavel Podivilov <podivilov@chromium.org>

        Reviewed by Pavel Feldman.

        Web Inspector: breakpoint is not disabled when clicking on breakpoints sidebar pane checkbox.
        https://bugs.webkit.org/show_bug.cgi?id=51745

        * inspector/front-end/Breakpoint.js:
        (WebInspector.Breakpoint.prototype.set enabled):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@74798 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index ebd1cd7..6cca7d0 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -2,6 +2,16 @@
 
         Reviewed by Pavel Feldman.
 
+        Web Inspector: breakpoint is not disabled when clicking on breakpoints sidebar pane checkbox.
+        https://bugs.webkit.org/show_bug.cgi?id=51745
+
+        * inspector/front-end/Breakpoint.js:
+        (WebInspector.Breakpoint.prototype.set enabled):
+
+2010-12-30  Pavel Podivilov  <podivilov@chromium.org>
+
+        Reviewed by Pavel Feldman.
+
         Web Inspector: call stack shows "Paused on a JavaScript breakpoint" when stepping.
         https://bugs.webkit.org/show_bug.cgi?id=51748
 

diff --git a/WebCore/inspector/front-end/Breakpoint.js b/WebCore/inspector/front-end/Breakpoint.js
index dd0ce12..0a888d2 100644 (file)
--- a/WebCore/inspector/front-end/Breakpoint.js
+++ b/WebCore/inspector/front-end/Breakpoint.js
@@ -48,6 +48,14 @@ WebInspector.Breakpoint.prototype = {
         return this._enabled;
     },
 
+    set enabled(enabled)
+    {
+        if (this._enabled === enabled)
+            return;
+        this.remove();
+        WebInspector.debuggerModel.setBreakpoint(this.sourceID, this.line, enabled, this.condition);
+    },
+
     get sourceText()
     {
         return this._sourceText;