DevTools: allow embedder handling remote debugger commands.

BUG=373325

Review URL: https://codereview.chromium.org/290873002

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

4 files changed, 102 insertions, 6 deletions

diff --git a/chrome/browser/devtools/chrome_devtools_manager_delegate.cc b/chrome/browser/devtools/chrome_devtools_manager_delegate.cc
index b2cd1ee..9b1f08b 100644
--- a/chrome/browser/devtools/chrome_devtools_manager_delegate.cc
+++ b/chrome/browser/devtools/chrome_devtools_manager_delegate.cc
@@ -26,3 +26,9 @@ void ChromeDevToolsManagerDelegate::Inspect(
     DevToolsWindow::OpenDevToolsWindowForWorker(profile, agent_host);
 #endif
 }
+
+base::DictionaryValue* ChromeDevToolsManagerDelegate::HandleCommand(
+    content::DevToolsAgentHost* agent_host,
+    base::DictionaryValue* command_dict) {
+  return NULL;
+}
diff --git a/chrome/browser/devtools/chrome_devtools_manager_delegate.h b/chrome/browser/devtools/chrome_devtools_manager_delegate.h
index f1bed42..4de8369 100644
--- a/chrome/browser/devtools/chrome_devtools_manager_delegate.h
+++ b/chrome/browser/devtools/chrome_devtools_manager_delegate.h
@@ -17,6 +17,9 @@ class ChromeDevToolsManagerDelegate : public content::DevToolsManagerDelegate {
   // content::DevToolsManagerDelegate overrides:
   virtual void Inspect(content::BrowserContext* browser_context,
                        content::DevToolsAgentHost* agent_host) OVERRIDE;
+  virtual base::DictionaryValue* HandleCommand(
+      content::DevToolsAgentHost* agent_host,
+      base::DictionaryValue* command_dict) OVERRIDE;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ChromeDevToolsManagerDelegate);
diff --git a/chrome/browser/devtools/devtools_protocol.cc b/chrome/browser/devtools/devtools_protocol.cc
index f14864c..ef7cb6a 100644
--- a/chrome/browser/devtools/devtools_protocol.cc
+++ b/chrome/browser/devtools/devtools_protocol.cc
@@ -6,13 +6,23 @@
 
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
+#include "base/strings/stringprintf.h"
 
 namespace {
+
+const char kErrorCodeParam[] = "code";
+const char kErrorParam[] = "error";
+const char kErrorMessageParam[] = "message";
 const char kIdParam[] = "id";
 const char kMethodParam[] = "method";
 const char kParamsParam[] = "params";
-const char kErrorParam[] = "error";
-const char kErrorCodeParam[] = "code";
+const char kResultParam[] = "result";
+
+// JSON RPC 2.0 spec: http://www.jsonrpc.org/specification#error_object
+enum Error {
+  kErrorInvalidParams = -32602
+};
+
 }  // namespace
 
 DevToolsProtocol::Message::~Message() {
@@ -46,6 +56,20 @@ std::string DevToolsProtocol::Command::Serialize() {
   return json_command;
 }
 
+scoped_ptr<DevToolsProtocol::Response>
+DevToolsProtocol::Command::SuccessResponse(base::DictionaryValue* result) {
+  return scoped_ptr<DevToolsProtocol::Response>(
+      new DevToolsProtocol::Response(id_, result));
+}
+
+scoped_ptr<DevToolsProtocol::Response>
+DevToolsProtocol::Command::InvalidParamResponse(const std::string& param) {
+  std::string message =
+      base::StringPrintf("Missing or invalid '%s' parameter", param.c_str());
+  return scoped_ptr<DevToolsProtocol::Response>(
+      new DevToolsProtocol::Response(id_, kErrorInvalidParams, message));
+}
+
 DevToolsProtocol::Notification::~Notification() {
 }
 
@@ -57,9 +81,55 @@ DevToolsProtocol::Notification::Notification(const std::string& method,
 DevToolsProtocol::Response::~Response() {
 }
 
-DevToolsProtocol::Response::Response(int id, int error_code)
+DevToolsProtocol::Response::Response(int id,
+                                     int error_code,
+                                     const std::string error_message)
+    : id_(id),
+      error_code_(error_code),
+      error_message_(error_message) {
+}
+
+DevToolsProtocol::Response::Response(int id, base::DictionaryValue* result)
     : id_(id),
-      error_code_(error_code) {
+      error_code_(0),
+      result_(result) {
+}
+
+base::DictionaryValue* DevToolsProtocol::Response::Serialize() {
+  base::DictionaryValue* response = new base::DictionaryValue();
+
+  response->SetInteger(kIdParam, id_);
+
+  if (error_code_) {
+    base::DictionaryValue* error_object = new base::DictionaryValue();
+    response->Set(kErrorParam, error_object);
+    error_object->SetInteger(kErrorCodeParam, error_code_);
+    if (!error_message_.empty())
+      error_object->SetString(kErrorMessageParam, error_message_);
+  } else if (result_) {
+    response->Set(kResultParam, result_->DeepCopy());
+  }
+
+  return response;
+}
+
+// static
+DevToolsProtocol::Command* DevToolsProtocol::ParseCommand(
+    base::DictionaryValue* command_dict) {
+  if (!command_dict)
+    return NULL;
+
+  int id;
+  if (!command_dict->GetInteger(kIdParam, &id) || id < 0)
+    return NULL;
+
+  std::string method;
+  if (!command_dict->GetString(kMethodParam, &method))
+    return NULL;
+
+  base::DictionaryValue* params = NULL;
+  command_dict->GetDictionary(kParamsParam, &params);
+  return new Command(id, method, params ? params->DeepCopy() : NULL);
 }
 
 // static
@@ -98,5 +168,5 @@ DevToolsProtocol::Response* DevToolsProtocol::ParseResponse(
   base::DictionaryValue* error_dict = NULL;
   if (dict->GetDictionary(kErrorParam, &error_dict))
     error_dict->GetInteger(kErrorCodeParam, &error_code);
-  return new Response(id, error_code);
+  return new Response(id, error_code, std::string());
 }
diff --git a/chrome/browser/devtools/devtools_protocol.h b/chrome/browser/devtools/devtools_protocol.h
index 9b15b9e..2e65810 100644
--- a/chrome/browser/devtools/devtools_protocol.h
+++ b/chrome/browser/devtools/devtools_protocol.h
@@ -16,6 +16,8 @@
 // from chrome component (see content/browser/devtools/devtools_protocol.*).
 class DevToolsProtocol {
  public:
+  class Response;
+
   class Message {
    public:
     virtual ~Message();
@@ -43,6 +45,12 @@ class DevToolsProtocol {
     int id() { return id_; }
     std::string Serialize();
 
+    // Creates success response. Takes ownership of |result|.
+    scoped_ptr<Response> SuccessResponse(base::DictionaryValue* result);
+
+    // Creates error response.
+    scoped_ptr<Response> InvalidParamResponse(const std::string& param);
+
    private:
     int id_;
 
@@ -56,12 +64,18 @@ class DevToolsProtocol {
     int id() { return id_; }
     int error_code() { return error_code_; }
 
+    // Result ownership is passed to the caller.
+    base::DictionaryValue* Serialize();
+
    private:
     friend class DevToolsProtocol;
 
-    Response(int id, int error_code);
+    Response(int id, int error_code, const std::string error_message);
+    Response(int id, base::DictionaryValue* result);
     int id_;
     int error_code_;
+    std::string error_message_;
+    scoped_ptr<base::DictionaryValue> result_;
 
     DISALLOW_COPY_AND_ASSIGN(Response);
   };
@@ -81,6 +95,9 @@ class DevToolsProtocol {
   };
 
   // Result ownership is passed to the caller.
+  static Command* ParseCommand(base::DictionaryValue* command_dict);
+
+  // Result ownership is passed to the caller.
   static Notification* ParseNotification(const std::string& json);
 
   // Result ownership is passed to the caller.