[1/5 chromium] Shows the clients which are controlled by ServiceWorker in DevTools.

content/browser/devtools/protocol/service_worker_handler.cc

Index: content/browser/devtools/protocol/service_worker_handler.cc
diff --git a/content/browser/devtools/protocol/service_worker_handler.cc b/content/browser/devtools/protocol/service_worker_handler.cc
index 907d8a5f3439f9fee30b402a6975b33dbaa673c2..b1a772f4935c392110947f4d16cc5257fb801fdd 100644
--- a/content/browser/devtools/protocol/service_worker_handler.cc
+++ b/content/browser/devtools/protocol/service_worker_handler.cc
@@ -397,6 +397,21 @@ Response ServiceWorkerHandler::DeliverPushMessage(
   return Response::OK();
 }
 
+Response ServiceWorkerHandler::GetClientInfo(DevToolsCommandId command_id,
+                                             const std::string& client_id) {
+  return Response::InternalError("Not implemented yet");

[pfeldman, 2015/06/05 12:19:54]

Why do you need command_id?

[horo, 2015/06/08 03:21:53]

"getTargetInfo" doesn't need to be "async" command.
It should be "sync" command like "GetTargetInfo(const std::string& target_id,
scoped_refptr<TargetInfo>* target_info)".
But I can't use TargetInfo type in service_worker_handler.* before it will be
defined in protocol.json and be used in a command which handler is "browser".

So (1) this cl introduces a dummy "async" method in service_worker_handler.*,
and (2) https://codereview.chromium.org/1151993003/ defines TargetInfo and an
"async" command in protocol.json, and (3)
https://codereview.chromium.org/1149383004/ introduces "sync" command in
service_worker_handler.*, and (4) https://codereview.chromium.org/1164583002/
removes '"async": true' from protocol.json, and (5)
https://codereview.chromium.org/1143363009/ removes "async" method from
service_worker_handler.*.

+}
+
+Response ServiceWorkerHandler::FocusClient(const std::string& client_id) {
+  scoped_refptr<DevToolsAgentHost> agent_host(
+      DevToolsAgentHost::GetForId(client_id));
+  if (!agent_host) {

[pfeldman, 2015/06/05 12:19:54]

drop {}

[horo, 2015/06/08 03:21:53]

Done.

+    return Response::InvalidParams("clientId");
+  }
+  agent_host->Activate();
+  return Response::OK();
+}
+
 void ServiceWorkerHandler::OpenNewDevToolsWindow(int process_id,
                                                  int devtools_agent_route_id) {
   scoped_refptr<DevToolsAgentHostImpl> agent_host(