Changeset 175993 in webkit

Timestamp:

Nov 11, 2014, 5:05:58 PM (12 years ago)

Author:

commit-queue@webkit.org

Message:

Web Inspector: Don't require a debugger be attached for inspector auto attach
​https://bugs.webkit.org/show_bug.cgi?id=138638

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

• inspector/remote/RemoteInspector.mm:

(Inspector::RemoteInspector::updateDebuggableAutomaticInspectCandidate):

Location:

trunk/Source/JavaScriptCore

Files:

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

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2014-11-11  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Web Inspector: Don't require a debugger be attached for inspector auto attach
+        https://bugs.webkit.org/show_bug.cgi?id=138638
+
+        Reviewed by Timothy Hatcher.
+
+        * inspector/remote/RemoteInspector.mm:
+        (Inspector::RemoteInspector::updateDebuggableAutomaticInspectCandidate):
+
 2014-11-11  Akos Kiss  <akiss@inf.u-szeged.hu>
 

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

@@ -190 +190 @@
         ASSERT_UNUSED(result, !result.isNewEntry);
 
-        // Don't allow automatic inspection unless there is a debugger or we are stopped.
-        if (!WTFIsDebuggerAttached() || !m_automaticInspectionEnabled || !m_enabled) {
+        // Don't allow automatic inspection unless it is allowed or we are stopped.
+        if (!m_automaticInspectionEnabled || !m_enabled) {
             pushListingSoon();
             return;