Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(318)

Side by Side Diff: content/browser/devtools/service_worker_devtools_agent_host.cc

Issue 2408133004: [DevTools] Implement Target.setDiscoverTargets method. (Closed)
Patch Set: new fancy range-based iteration in observer list! Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/devtools/service_worker_devtools_agent_host.h" 5 #include "content/browser/devtools/service_worker_devtools_agent_host.h"
6 6
7 #include "base/strings/stringprintf.h" 7 #include "base/strings/stringprintf.h"
8 #include "content/browser/devtools/devtools_manager.h"
8 #include "content/browser/devtools/devtools_protocol_handler.h" 9 #include "content/browser/devtools/devtools_protocol_handler.h"
9 #include "content/browser/devtools/protocol/devtools_protocol_dispatcher.h" 10 #include "content/browser/devtools/protocol/devtools_protocol_dispatcher.h"
10 #include "content/browser/devtools/service_worker_devtools_manager.h" 11 #include "content/browser/devtools/service_worker_devtools_manager.h"
11 #include "content/browser/service_worker/service_worker_context_core.h" 12 #include "content/browser/service_worker/service_worker_context_core.h"
12 #include "content/browser/service_worker/service_worker_version.h" 13 #include "content/browser/service_worker/service_worker_version.h"
13 #include "content/public/browser/browser_thread.h" 14 #include "content/public/browser/browser_thread.h"
14 #include "content/public/browser/render_process_host.h" 15 #include "content/public/browser/render_process_host.h"
15 16
16 namespace content { 17 namespace content {
17 18
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 WorkerId worker_id, 57 WorkerId worker_id,
57 const ServiceWorkerIdentifier& service_worker, 58 const ServiceWorkerIdentifier& service_worker,
58 bool is_installed_version) 59 bool is_installed_version)
59 : WorkerDevToolsAgentHost(worker_id), 60 : WorkerDevToolsAgentHost(worker_id),
60 service_worker_(new ServiceWorkerIdentifier(service_worker)), 61 service_worker_(new ServiceWorkerIdentifier(service_worker)),
61 network_handler_(new devtools::network::NetworkHandler()), 62 network_handler_(new devtools::network::NetworkHandler()),
62 version_installed_time_(is_installed_version ? base::Time::Now() 63 version_installed_time_(is_installed_version ? base::Time::Now()
63 : base::Time()) { 64 : base::Time()) {
64 DevToolsProtocolDispatcher* dispatcher = protocol_handler()->dispatcher(); 65 DevToolsProtocolDispatcher* dispatcher = protocol_handler()->dispatcher();
65 dispatcher->SetNetworkHandler(network_handler_.get()); 66 dispatcher->SetNetworkHandler(network_handler_.get());
67 DevToolsManager::GetInstance()->AgentHostCreated(this);
66 } 68 }
67 69
68 std::string ServiceWorkerDevToolsAgentHost::GetType() { 70 std::string ServiceWorkerDevToolsAgentHost::GetType() {
69 return kTypeServiceWorker; 71 return kTypeServiceWorker;
70 } 72 }
71 73
72 std::string ServiceWorkerDevToolsAgentHost::GetTitle() { 74 std::string ServiceWorkerDevToolsAgentHost::GetTitle() {
73 if (RenderProcessHost* host = RenderProcessHost::FromID(worker_id().first)) { 75 if (RenderProcessHost* host = RenderProcessHost::FromID(worker_id().first)) {
74 return base::StringPrintf("Worker pid:%d", 76 return base::StringPrintf("Worker pid:%d",
75 base::GetProcId(host->GetHandle())); 77 base::GetProcId(host->GetHandle()));
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
128 } 130 }
129 131
130 bool ServiceWorkerDevToolsAgentHost::Matches( 132 bool ServiceWorkerDevToolsAgentHost::Matches(
131 const ServiceWorkerIdentifier& other) { 133 const ServiceWorkerIdentifier& other) {
132 return service_worker_->Matches(other); 134 return service_worker_->Matches(other);
133 } 135 }
134 136
135 ServiceWorkerDevToolsAgentHost::~ServiceWorkerDevToolsAgentHost() { 137 ServiceWorkerDevToolsAgentHost::~ServiceWorkerDevToolsAgentHost() {
136 ServiceWorkerDevToolsManager::GetInstance()->RemoveInspectedWorkerData( 138 ServiceWorkerDevToolsManager::GetInstance()->RemoveInspectedWorkerData(
137 worker_id()); 139 worker_id());
140 DevToolsManager::GetInstance()->AgentHostDestroyed(this);
138 } 141 }
139 142
140 } // namespace content 143 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698