DevTools: Added service workers to remote debugging targets

content/browser/devtools/embedded_worker_devtools_agent_host.cc

Index: content/browser/devtools/embedded_worker_devtools_agent_host.cc
diff --git a/content/browser/devtools/embedded_worker_devtools_agent_host.cc b/content/browser/devtools/embedded_worker_devtools_agent_host.cc
index 9f604001134b61ba4a11c8e40974e723e8b516e5..528ffcf157aabc2956666e74c7f082f66199c542 100644
--- a/content/browser/devtools/embedded_worker_devtools_agent_host.cc
+++ b/content/browser/devtools/embedded_worker_devtools_agent_host.cc
@@ -4,9 +4,11 @@
 
 #include "content/browser/devtools/embedded_worker_devtools_agent_host.h"
 
+#include "base/strings/utf_string_conversions.h"
 #include "content/browser/devtools/devtools_manager_impl.h"
 #include "content/browser/devtools/devtools_protocol.h"
 #include "content/browser/devtools/devtools_protocol_constants.h"
+#include "content/browser/shared_worker/shared_worker_service_impl.h"
 #include "content/common/devtools_messages.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_process_host.h"
@@ -38,6 +40,30 @@ bool EmbeddedWorkerDevToolsAgentHost::IsWorker() const {
   return true;
 }
 
+std::string EmbeddedWorkerDevToolsAgentHost::GetType() {
+  return shared_worker_ ? kTypeWorker : kTypeServiceWorker;
+}
+
+std::string EmbeddedWorkerDevToolsAgentHost::GetTitle() {
+  return shared_worker_ ? base::UTF16ToUTF8(shared_worker_->name()) : "";
+}
+
+GURL EmbeddedWorkerDevToolsAgentHost::GetURL() {
+  return shared_worker_ ? shared_worker_->url() : GURL();

[dgozman, 2014/08/13 15:57:02]

Can we get URL for shared worker?

[vkuzkokov, 2014/08/14 07:48:17]

Service worker, you mean? It requires changes in EmbeddedWorkerInstance which
belong in another CL.

[dgozman, 2014/08/14 08:15:33]

Can we make those changes beforehand? Otherwise you will get disfunctional Close
button as well as no way to differentiate between service workers. I think we
should not expose not yet ready service workers then.

+}
+
+bool EmbeddedWorkerDevToolsAgentHost::Activate() {
+  return false;
+}
+
+bool EmbeddedWorkerDevToolsAgentHost::Close() {
+  if (shared_worker_) {
+    return SharedWorkerServiceImpl::GetInstance()->TerminateWorker(
+        worker_id_.first, worker_id_.second);
+  }
+  return false;

[dgozman, 2014/08/13 15:57:02]

Why can't we close shared worker?

[vkuzkokov, 2014/08/14 07:48:17]

Ditto.

+}
+
 void EmbeddedWorkerDevToolsAgentHost::SendMessageToAgent(
     IPC::Message* message_raw) {
   scoped_ptr<IPC::Message> message(message_raw);