Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Unified Diff: Source/web/WebDevToolsAgentImpl.cpp

Issue 932523003: DevTools: [Blink-side] Fix debugger command order (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Add m_attached check Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/web/WebDevToolsAgentImpl.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/web/WebDevToolsAgentImpl.cpp
diff --git a/Source/web/WebDevToolsAgentImpl.cpp b/Source/web/WebDevToolsAgentImpl.cpp
index d711f645e1008e0f7f2857c37434495d4bfe8aca..fd1a0e280cbf0fe6f107503dfea3ac325ebcac3e 100644
--- a/Source/web/WebDevToolsAgentImpl.cpp
+++ b/Source/web/WebDevToolsAgentImpl.cpp
@@ -190,8 +190,13 @@ public:
virtual ~DebuggerTask() { }
virtual void run()
{
- if (WebDevToolsAgent* webagent = m_descriptor->agent())
- webagent->dispatchOnInspectorBackend(m_descriptor->message());
+ WebDevToolsAgent* webagent = m_descriptor->agent();
+ if (!webagent)
+ return;
+
+ WebDevToolsAgentImpl* agentImpl = static_cast<WebDevToolsAgentImpl*>(webagent);
+ if (agentImpl->m_attached)
+ agentImpl->inspectorController()->dispatchMessageFromFrontend(m_descriptor->message());
}
private:
@@ -444,7 +449,10 @@ void WebDevToolsAgentImpl::dispatchOnInspectorBackend(const WebString& message)
{
if (!m_attached)
return;
- inspectorController()->dispatchMessageFromFrontend(message);
+ if (WebDevToolsAgent::shouldInterruptForMessage(message))
+ PageScriptDebugServer::shared().runPendingTasks();
+ else
+ inspectorController()->dispatchMessageFromFrontend(message);
}
void WebDevToolsAgentImpl::inspectElementAt(const WebPoint& point)
« no previous file with comments | « Source/web/WebDevToolsAgentImpl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698