Changeset 161953 in webkit for trunk/Source/JavaScriptCore

Timestamp:

Jan 14, 2014, 12:35:40 AM (11 years ago)

Author:

[email protected]

Message:

Web Inspector: RemoteInspector::updateDebuggable may miss a push
​https://bugs.webkit.org/show_bug.cgi?id=126965

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2014-01-14
Reviewed by Timothy Hatcher.

• inspector/remote/RemoteInspector.mm:

(Inspector::RemoteInspector::updateDebuggable):
Always push an update. If a debuggable went from allowed to
not allowed, we would have missed pushing an update.

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/remote/RemoteInspector.mm (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2014-01-14  Joseph Pecoraro  <[email protected]>
+
+        Web Inspector: RemoteInspector::updateDebuggable may miss a push
+        https://bugs.webkit.org/show_bug.cgi?id=126965
+
+        Reviewed by Timothy Hatcher.
+
+        * inspector/remote/RemoteInspector.mm:
+        (Inspector::RemoteInspector::updateDebuggable):
+        Always push an update. If a debuggable went from allowed to
+        not allowed, we would have missed pushing an update.
+
 2014-01-13  Mark Hahnenberg  <[email protected]>
 

trunk/Source/JavaScriptCore/inspector/remote/RemoteInspector.mm

@@ -132 +132 @@
     ASSERT_UNUSED(result, !result.isNewEntry);
 
-    if (debuggable->remoteDebuggingAllowed())
-        pushListingSoon();
+    pushListingSoon();
 }