DevTools: [Blink-side] Fix debugger command order

Source/web/WebDevToolsAgentImpl.cpp

 /*
  * Copyright (C) 2010-2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 173 matching lines @@
 public:
     DebuggerTask(PassOwnPtr<WebDevToolsAgent::MessageDescriptor> descriptor)
         : m_descriptor(descriptor)
     {
     }
 
     virtual ~DebuggerTask() { }
     virtual void run()
     {
         if (WebDevToolsAgent* webagent = m_descriptor->agent())
-            webagent->dispatchOnInspectorBackend(m_descriptor->message());
+            webagent->dispatchOnInspectorBackend(m_descriptor->message(), true);
     }
 
 private:
     OwnPtr<WebDevToolsAgent::MessageDescriptor> m_descriptor;
 };
 
 WebDevToolsAgentImpl::WebDevToolsAgentImpl(
     WebViewImpl* webViewImpl,
     WebDevToolsAgentClient* client)
     : m_layerTreeId(0)
@@ skipping 228 matching lines @@
 {
     if (!m_webViewImpl->page()->focusController().isFocused())
         m_webViewImpl->setFocus(true);
 
     m_generatingEvent = true;
     WebMouseEvent webEvent = WebMouseEventBuilder(m_webViewImpl->mainFrameImpl()->frameView(), event);
     m_webViewImpl->handleInputEvent(webEvent);
     m_generatingEvent = false;
 }
 
-void WebDevToolsAgentImpl::dispatchOnInspectorBackend(const WebString& message)
+void WebDevToolsAgentImpl::dispatchOnInspectorBackend(const WebString& message, bool ignoreInterrupt)
 {
     if (!m_attached)
         return;
-    inspectorController()->dispatchMessageFromFrontend(message);
+    if (ignoreInterrupt || !WebDevToolsAgent::shouldInterruptForMessage(message))

[yurys, 2015/02/16 16:33:07]

You don't need this parameter as before the chromium-side patch is landed
WebDevToolsAgent::shouldInterruptForMessage must always return false
(corresponding command should have been routed into v8 queue) and after it is
landed you will start receiving those commands here. Or am I missing something?

[yurys, 2015/02/16 16:56:15]

To clarify this after offline discussion. We can make DebuggerTask a friend of
WebDevToolsAgentImpl and 
inspectorController()->dispatchMessageFromFrontend(message); from it (it will
require static_cast<WebDevToolsAgentImpl*> but that's fine there).

[sergeyv, 2015/02/16 17:30:13]

Done.

+        inspectorController()->dispatchMessageFromFrontend(message);
+    else
+        PageScriptDebugServer::shared().runPendingTasks();
 }
 
 void WebDevToolsAgentImpl::inspectElementAt(const WebPoint& point)
 {
     m_webViewImpl->inspectElementAt(point);
 }
 
 InspectorController* WebDevToolsAgentImpl::inspectorController()
 {
     if (Page* page = m_webViewImpl->page())
@@ skipping 118 matching lines @@
         || commandName == InspectorBackendDispatcher::commandName(InspectorBackendDispatcher::kDebugger_removeBreakpointCmd)
         || commandName == InspectorBackendDispatcher::commandName(InspectorBackendDispatcher::kDebugger_setBreakpointsActiveCmd);
 }
 
 void WebDevToolsAgent::processPendingMessages()
 {
     PageScriptDebugServer::shared().runPendingTasks();
 }
 
 } // namespace blink