[webkit-reviews] review requested: [Bug 39302] [Chromium] WebDevToolsAgentClient::sendMessageToFrontendOnIOThread needs to be implemented as virtual function : [Attachment 58880] will not break functionality, but requires 4 steps to commit

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 16 04:52:28 PDT 2010


Mikhail Naganov <mnaganov at chromium.org> has asked  for review:
Bug 39302: [Chromium] WebDevToolsAgentClient::sendMessageToFrontendOnIOThread
needs to be implemented as virtual function
https://bugs.webkit.org/show_bug.cgi?id=39302

Attachment 58880: will not break functionality, but requires 4 steps to commit
https://bugs.webkit.org/attachment.cgi?id=58880&action=review

------- Additional Comments from Mikhail Naganov <mnaganov at chromium.org>
Chromium part:


diff --git a/chrome/renderer/devtools_agent_filter.cc
b/chrome/renderer/devtools_agent_filter.cc
index 5eb54ed..4c98cbe 100644
--- a/chrome/renderer/devtools_agent_filter.cc
+++ b/chrome/renderer/devtools_agent_filter.cc
@@ -64,8 +64,21 @@ void DevToolsAgentFilter::OnDebuggerPauseScript() {
   WebDevToolsAgent::debuggerPauseScript();
 }
 
+namespace {
+
+class WebDevToolsMessageTransportImpl : public
WebKit::WebDevToolsMessageTransport {
+ public:
+  void sendMessageToFrontendOnIOThread(const WebKit::WebDevToolsMessageData&
data) {
+    DevToolsAgentFilter::SendRpcMessage(DevToolsMessageData(data));
+  }
+};
+
+}  // namespace
+
 void DevToolsAgentFilter::OnRpcMessage(const DevToolsMessageData& data) {
+  WebDevToolsMessageTransportImpl transport;
   message_handled_ = WebDevToolsAgent::dispatchMessageFromFrontendOnIOThread(
+      &transport,
       data.ToWebDevToolsMessageData());
 }


More information about the webkit-reviews mailing list