Revert "DevTools: use WebDevToolsMessageData for devtools rpc messages"

This reverts commit ea393f34b293ebf8b96bef5855b9f41faf82b192.

Review URL: http://codereview.chromium.org/492018

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34334 0039d316-1c4b-4281-b951-d872f2087c98

6 files changed, 105 insertions, 36 deletions

diff --git a/chrome/renderer/devtools_agent.cc b/chrome/renderer/devtools_agent.cc
index 11211fd..d1e81e2 100644
--- a/chrome/renderer/devtools_agent.cc
+++ b/chrome/renderer/devtools_agent.cc
@@ -9,16 +9,13 @@
 #include "chrome/renderer/devtools_agent_filter.h"
 #include "chrome/renderer/render_view.h"
 #include "third_party/WebKit/WebKit/chromium/public/WebDevToolsAgent.h"
-#include "third_party/WebKit/WebKit/chromium/public/WebDevToolsMessageData.h"
 #include "third_party/WebKit/WebKit/chromium/public/WebPoint.h"
 #include "third_party/WebKit/WebKit/chromium/public/WebString.h"
-#include "webkit/glue/devtools/devtools_message_data.h"
 #include "webkit/glue/glue_util.h"
 
 using WebKit::WebDevToolsAgent;
 using WebKit::WebPoint;
 using WebKit::WebString;
-using WebKit::WebVector;
 using WebKit::WebView;
 
 // static
@@ -56,11 +53,19 @@ bool DevToolsAgent::OnMessageReceived(const IPC::Message& message) {
   return handled;
 }
 
-void DevToolsAgent::sendMessageToFrontend(
-    const WebKit::WebDevToolsMessageData& data) {
+void DevToolsAgent::sendMessageToFrontend(const WebString& class_name,
+                                          const WebString& method_name,
+                                          const WebString& param1,
+                                          const WebString& param2,
+                                          const WebString& param3) {
   IPC::Message* m = new ViewHostMsg_ForwardToDevToolsClient(
       routing_id_,
-      DevToolsClientMsg_RpcMessage(DevToolsMessageData(data)));
+      DevToolsClientMsg_RpcMessage(
+          class_name.utf8(),
+          method_name.utf8(),
+          param1.utf8(),
+          param2.utf8(),
+          param3.utf8()));
   render_view_->Send(m);
 }
 
@@ -108,10 +113,19 @@ void DevToolsAgent::OnDetach() {
   }
 }
 
-void DevToolsAgent::OnRpcMessage(const DevToolsMessageData& data) {
+void DevToolsAgent::OnRpcMessage(const std::string& class_name,
+                                 const std::string& method_name,
+                                 const std::string& param1,
+                                 const std::string& param2,
+                                 const std::string& param3) {
   WebDevToolsAgent* web_agent = GetWebAgent();
   if (web_agent) {
-    web_agent->dispatchMessageFromFrontend(data.ToWebDevToolsMessageData());
+    web_agent->dispatchMessageFromFrontend(
+        WebString::fromUTF8(class_name),
+        WebString::fromUTF8(method_name),
+        WebString::fromUTF8(param1),
+        WebString::fromUTF8(param2),
+        WebString::fromUTF8(param3));
   }
 }
 
@@ -141,6 +155,15 @@ WebDevToolsAgent* DevToolsAgent::GetWebAgent() {
 
 // static
 void WebKit::WebDevToolsAgentClient::sendMessageToFrontendOnIOThread(
-    const WebDevToolsMessageData& data) {
-    DevToolsAgentFilter::SendRpcMessage(DevToolsMessageData(data));
+    const WebString& class_name,
+    const WebString& method_name,
+    const WebString& param1,
+    const WebString& param2,
+    const WebString& param3) {
+    DevToolsAgentFilter::SendRpcMessage(
+        class_name.utf8(),
+        method_name.utf8(),
+        param1.utf8(),
+        param2.utf8(),
+        param3.utf8());
 }
diff --git a/chrome/renderer/devtools_agent.h b/chrome/renderer/devtools_agent.h
index 4d05fe0..26a6297 100644
--- a/chrome/renderer/devtools_agent.h
+++ b/chrome/renderer/devtools_agent.h
@@ -21,7 +21,6 @@ class WebDevToolsAgent;
 }
 
 class RenderView;
-struct DevToolsMessageData;
 
 // DevToolsAgent belongs to the inspectable RenderView and provides Glue's
 // agents with the communication capabilities. All messages from/to Glue's
@@ -38,9 +37,11 @@ class DevToolsAgent : public WebKit::WebDevToolsAgentClient {
   virtual bool OnMessageReceived(const IPC::Message& message);
 
   // WebDevToolsAgentClient implementation
-  virtual void sendMessageToFrontend(
-      const WebKit::WebDevToolsMessageData& data);
-
+  virtual void sendMessageToFrontend(const WebKit::WebString& class_name,
+                                     const WebKit::WebString& method_name,
+                                     const WebKit::WebString& param1,
+                                     const WebKit::WebString& param2,
+                                     const WebKit::WebString& param3);
   virtual int hostIdentifier();
   virtual void forceRepaint();
   virtual void runtimeFeatureStateChanged(const WebKit::WebString& feature,
@@ -58,7 +59,11 @@ class DevToolsAgent : public WebKit::WebDevToolsAgentClient {
 
   void OnAttach(const std::vector<std::string>& runtime_features);
   void OnDetach();
-  void OnRpcMessage(const DevToolsMessageData& data);
+  void OnRpcMessage(const std::string& class_name,
+                    const std::string& method_name,
+                    const std::string& param1,
+                    const std::string& param2,
+                    const std::string& param3);
   void OnInspectElement(int x, int y);
   void OnSetApuAgentEnabled(bool enabled);
 
diff --git a/chrome/renderer/devtools_agent_filter.cc b/chrome/renderer/devtools_agent_filter.cc
index d3a8289..375c743 100644
--- a/chrome/renderer/devtools_agent_filter.cc
+++ b/chrome/renderer/devtools_agent_filter.cc
@@ -11,9 +11,7 @@
 #include "chrome/renderer/plugin_channel_host.h"
 #include "chrome/renderer/render_view.h"
 #include "third_party/WebKit/WebKit/chromium/public/WebDevToolsAgent.h"
-#include "third_party/WebKit/WebKit/chromium/public/WebDevToolsMessageData.h"
 #include "third_party/WebKit/WebKit/chromium/public/WebString.h"
-#include "webkit/glue/devtools/devtools_message_data.h"
 
 using WebKit::WebDevToolsAgent;
 using WebKit::WebString;
@@ -64,15 +62,32 @@ void DevToolsAgentFilter::OnDebuggerPauseScript() {
   WebDevToolsAgent::debuggerPauseScript();
 }
 
-void DevToolsAgentFilter::OnRpcMessage(const DevToolsMessageData& data) {
+void DevToolsAgentFilter::OnRpcMessage(const std::string& class_name,
+                                       const std::string& method_name,
+                                       const std::string& param1,
+                                       const std::string& param2,
+                                       const std::string& param3) {
   message_handled_ = WebDevToolsAgent::dispatchMessageFromFrontendOnIOThread(
-      data.ToWebDevToolsMessageData());
+      WebString::fromUTF8(class_name),
+      WebString::fromUTF8(method_name),
+      WebString::fromUTF8(param1),
+      WebString::fromUTF8(param2),
+      WebString::fromUTF8(param3));
 }
 
 // static
-void DevToolsAgentFilter::SendRpcMessage(const DevToolsMessageData& data) {
+void DevToolsAgentFilter::SendRpcMessage(const std::string& class_name,
+                                         const std::string& method_name,
+                                         const std::string& param1,
+                                         const std::string& param2,
+                                         const std::string& param3) {
   IPC::Message* m = new ViewHostMsg_ForwardToDevToolsClient(
       current_routing_id_,
-      DevToolsClientMsg_RpcMessage(data));
+      DevToolsClientMsg_RpcMessage(
+          class_name,
+          method_name,
+          param1,
+          param2,
+          param3));
   channel_->Send(m);
 }
diff --git a/chrome/renderer/devtools_agent_filter.h b/chrome/renderer/devtools_agent_filter.h
index 8d23918..ac5fd0c 100644
--- a/chrome/renderer/devtools_agent_filter.h
+++ b/chrome/renderer/devtools_agent_filter.h
@@ -10,8 +10,6 @@
 
 #include "ipc/ipc_channel_proxy.h"
 
-struct DevToolsMessageData;
-
 // DevToolsAgentFilter is registered as an IPC filter in order to be able to
 // dispatch messages while on the IO thread. The reason for that is that while
 // debugging, Render thread is being held by the v8 and hence no messages
@@ -24,7 +22,11 @@ class DevToolsAgentFilter : public IPC::ChannelProxy::MessageFilter {
   DevToolsAgentFilter();
   virtual ~DevToolsAgentFilter();
 
-  static void SendRpcMessage(const DevToolsMessageData& data);
+  static void SendRpcMessage(const std::string& class_name,
+                             const std::string& method_name,
+                             const std::string& param1,
+                             const std::string& param2,
+                             const std::string& param3);
 
  private:
   // IPC::ChannelProxy::MessageFilter override. Called on IO thread.
@@ -38,7 +40,11 @@ class DevToolsAgentFilter : public IPC::ChannelProxy::MessageFilter {
   // handle debug messages even when v8 is stopped.
   void OnDebuggerCommand(const std::string& command);
   void OnDebuggerPauseScript();
-  void OnRpcMessage(const DevToolsMessageData& data);
+  void OnRpcMessage(const std::string& class_name,
+                    const std::string& method_name,
+                    const std::string& param1,
+                    const std::string& param2,
+                    const std::string& param3);
 
   bool message_handled_;
 
diff --git a/chrome/renderer/devtools_client.cc b/chrome/renderer/devtools_client.cc
index 2ab162e..8083288 100644
--- a/chrome/renderer/devtools_client.cc
+++ b/chrome/renderer/devtools_client.cc
@@ -11,7 +11,6 @@
 #include "chrome/renderer/render_thread.h"
 #include "chrome/renderer/render_view.h"
 #include "third_party/WebKit/WebKit/chromium/public/WebDevToolsFrontend.h"
-#include "third_party/WebKit/WebKit/chromium/public/WebDevToolsMessageData.h"
 #include "third_party/WebKit/WebKit/chromium/public/WebString.h"
 
 using WebKit::WebDevToolsFrontend;
@@ -48,9 +47,17 @@ bool DevToolsClient::OnMessageReceived(const IPC::Message& message) {
   return handled;
 }
 
-void DevToolsClient::sendMessageToAgent(
-      const WebKit::WebDevToolsMessageData& data) {
-  Send(DevToolsAgentMsg_RpcMessage(DevToolsMessageData(data)));
+void DevToolsClient::sendMessageToAgent(const WebString& class_name,
+                                        const WebString& method_name,
+                                        const WebString& param1,
+                                        const WebString& param2,
+                                        const WebString& param3) {
+  Send(DevToolsAgentMsg_RpcMessage(
+      class_name.utf8(),
+      method_name.utf8(),
+      param1.utf8(),
+      param2.utf8(),
+      param3.utf8()));
 }
 
 void DevToolsClient::sendDebuggerCommandToAgent(const WebString& command) {
@@ -81,7 +88,15 @@ void DevToolsClient::undockWindow() {
       render_view_->routing_id()));
 }
 
-void DevToolsClient::OnRpcMessage(const DevToolsMessageData& data) {
+void DevToolsClient::OnRpcMessage(const std::string& class_name,
+                                  const std::string& method_name,
+                                  const std::string& param1,
+                                  const std::string& param2,
+                                  const std::string& param3) {
   web_tools_frontend_->dispatchMessageFromAgent(
-      data.ToWebDevToolsMessageData());
+      WebString::fromUTF8(class_name),
+      WebString::fromUTF8(method_name),
+      WebString::fromUTF8(param1),
+      WebString::fromUTF8(param2),
+      WebString::fromUTF8(param3));
 }
diff --git a/chrome/renderer/devtools_client.h b/chrome/renderer/devtools_client.h
index c8eba9f..aad5c09 100644
--- a/chrome/renderer/devtools_client.h
+++ b/chrome/renderer/devtools_client.h
@@ -21,8 +21,6 @@ namespace WebKit {
 class WebDevToolsFrontend;
 }
 
-struct DevToolsMessageData;
-
 // Developer tools UI end of communication channel between the render process of
 // the page being inspected and tools UI renderer process. All messages will
 // go through browser process. On the side of the inspected page there's
@@ -39,8 +37,11 @@ class DevToolsClient : public WebKit::WebDevToolsFrontendClient {
   bool OnMessageReceived(const IPC::Message& message);
 
   // WebDevToolsFrontendClient implementation
-  virtual void sendMessageToAgent(
-      const WebKit::WebDevToolsMessageData& data);
+  virtual void sendMessageToAgent(const WebKit::WebString& class_name,
+                                  const WebKit::WebString& method_name,
+                                  const WebKit::WebString& param1,
+                                  const WebKit::WebString& param2,
+                                  const WebKit::WebString& param3);
   virtual void sendDebuggerCommandToAgent(const WebKit::WebString& command);
   virtual void sendDebuggerPauseScript();
 
@@ -50,7 +51,11 @@ class DevToolsClient : public WebKit::WebDevToolsFrontendClient {
   virtual void undockWindow();
 
  private:
-  void OnRpcMessage(const DevToolsMessageData& data);
+  void OnRpcMessage(const std::string& class_name,
+                    const std::string& method_name,
+                    const std::string& param1,
+                    const std::string& param2,
+                    const std::string& param3);
 
   // Sends message to DevToolsAgent.
   void Send(const IPC::Message& tools_agent_message);