Title: [175993] trunk/Source/_javascript_Core
- Revision
- 175993
- Author
- [email protected]
- Date
- 2014-11-11 17:05:58 -0800 (Tue, 11 Nov 2014)
Log 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 <[email protected]> on 2014-11-11
Reviewed by Timothy Hatcher.
* inspector/remote/RemoteInspector.mm:
(Inspector::RemoteInspector::updateDebuggableAutomaticInspectCandidate):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (175992 => 175993)
--- trunk/Source/_javascript_Core/ChangeLog 2014-11-12 00:59:54 UTC (rev 175992)
+++ trunk/Source/_javascript_Core/ChangeLog 2014-11-12 01:05:58 UTC (rev 175993)
@@ -1,3 +1,13 @@
+2014-11-11 Joseph Pecoraro <[email protected]>
+
+ 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 <[email protected]>
Handle cases in StackVisitor::Frame::existingArguments() when lexicalEnvironment and/or unmodifiedArgumentsRegister is not set up yet
Modified: trunk/Source/_javascript_Core/inspector/remote/RemoteInspector.mm (175992 => 175993)
--- trunk/Source/_javascript_Core/inspector/remote/RemoteInspector.mm 2014-11-12 00:59:54 UTC (rev 175992)
+++ trunk/Source/_javascript_Core/inspector/remote/RemoteInspector.mm 2014-11-12 01:05:58 UTC (rev 175993)
@@ -189,8 +189,8 @@
auto result = m_debuggableMap.set(identifier, std::make_pair(debuggable, debuggable->info()));
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;
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
