[DevTools] Used generated Inspector.detached in DevToolsHttpHandlerImpl

content/browser/devtools/devtools_http_handler_impl.cc

Index: content/browser/devtools/devtools_http_handler_impl.cc
diff --git a/content/browser/devtools/devtools_http_handler_impl.cc b/content/browser/devtools/devtools_http_handler_impl.cc
index f2017cbb83454d6150e64cf33cbd159afd2c637c..61ed53ba75ac5be51da8acc5daaa233797786332 100644
--- a/content/browser/devtools/devtools_http_handler_impl.cc
+++ b/content/browser/devtools/devtools_http_handler_impl.cc
@@ -349,23 +349,16 @@ class DevToolsAgentHostClientImpl : public DevToolsAgentHostClient {
   void AgentHostClosed(DevToolsAgentHost* agent_host,
                        bool replaced_with_another_client) override {
     DCHECK(agent_host == agent_host_.get());
-    agent_host_ = NULL;
 
     base::DictionaryValue notification;

[dgozman, 2014/11/20 14:29:14]

nit: notification unused.

[vkuzkokov, 2014/11/20 14:47:35]

Done.

-    notification.SetString(
-        devtools::Inspector::detached::kParamReason,
+    devtools::inspector::Client client(
+        base::Bind(&DevToolsAgentHostClientImpl::DispatchProtocolMessage,

[dgozman, 2014/11/20 14:29:14]

Let's extract this bound function and call it |raw_message_callback|.

[vkuzkokov, 2014/11/20 14:47:35]

Done.

+                   base::Unretained(this), base::Unretained(agent_host)));
+    client.Detached(devtools::inspector::DetachedParams::Create()->set_reason(

[dgozman, 2014/11/20 14:29:13]

|inspector.Detached| would be nicer.

[vkuzkokov, 2014/11/20 14:47:35]

Done.

         replaced_with_another_client ?
-            "replaced_with_devtools" : "target_closed");
-    std::string response = DevToolsProtocol::CreateNotification(
-        devtools::Inspector::detached::kName,
-        notification.DeepCopy())->Serialize();
-    message_loop_->PostTask(
-        FROM_HERE,
-        base::Bind(&ServerWrapper::SendOverWebSocket,
-                   base::Unretained(server_wrapper_),
-                   connection_id_,
-                   response));
+            "replaced_with_devtools" : "target_closed"));
 
+    agent_host_ = nullptr;
     message_loop_->PostTask(
         FROM_HERE,
         base::Bind(&ServerWrapper::Close,