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

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

Issue 1893313007: DevTools: Attach all Service Workers except for old redundant ones. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: incorporated pfeldman's comment Created 4 years, 8 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_protocol_handler.h" 8 #include "content/browser/devtools/devtools_protocol_handler.h"
9 #include "content/browser/devtools/protocol/devtools_protocol_dispatcher.h" 9 #include "content/browser/devtools/protocol/devtools_protocol_dispatcher.h"
10 #include "content/browser/devtools/service_worker_devtools_manager.h" 10 #include "content/browser/devtools/service_worker_devtools_manager.h"
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 if (ServiceWorkerContextCore* context = context_weak.get()) { 47 if (ServiceWorkerContextCore* context = context_weak.get()) {
48 if (ServiceWorkerVersion* version = context->GetLiveVersion(version_id)) 48 if (ServiceWorkerVersion* version = context->GetLiveVersion(version_id))
49 version->SetDevToolsAttached(attached); 49 version->SetDevToolsAttached(attached);
50 } 50 }
51 } 51 }
52 52
53 } // namespace 53 } // namespace
54 54
55 ServiceWorkerDevToolsAgentHost::ServiceWorkerDevToolsAgentHost( 55 ServiceWorkerDevToolsAgentHost::ServiceWorkerDevToolsAgentHost(
56 WorkerId worker_id, 56 WorkerId worker_id,
57 const ServiceWorkerIdentifier& service_worker) 57 const ServiceWorkerIdentifier& service_worker,
58 bool is_installed_version)
58 : WorkerDevToolsAgentHost(worker_id), 59 : WorkerDevToolsAgentHost(worker_id),
59 service_worker_(new ServiceWorkerIdentifier(service_worker)), 60 service_worker_(new ServiceWorkerIdentifier(service_worker)),
60 network_handler_(new devtools::network::NetworkHandler()) { 61 network_handler_(new devtools::network::NetworkHandler()),
62 version_installed_time_(is_installed_version ? base::Time::Now()
63 : base::Time()) {
61 DevToolsProtocolDispatcher* dispatcher = protocol_handler()->dispatcher(); 64 DevToolsProtocolDispatcher* dispatcher = protocol_handler()->dispatcher();
62 dispatcher->SetNetworkHandler(network_handler_.get()); 65 dispatcher->SetNetworkHandler(network_handler_.get());
63 } 66 }
64 67
65 DevToolsAgentHost::Type ServiceWorkerDevToolsAgentHost::GetType() { 68 DevToolsAgentHost::Type ServiceWorkerDevToolsAgentHost::GetType() {
66 return TYPE_SERVICE_WORKER; 69 return TYPE_SERVICE_WORKER;
67 } 70 }
68 71
69 std::string ServiceWorkerDevToolsAgentHost::GetTitle() { 72 std::string ServiceWorkerDevToolsAgentHost::GetTitle() {
70 if (RenderProcessHost* host = RenderProcessHost::FromID(worker_id().first)) { 73 if (RenderProcessHost* host = RenderProcessHost::FromID(worker_id().first)) {
(...skipping 27 matching lines...) Expand all
98 } 101 }
99 102
100 void ServiceWorkerDevToolsAgentHost::OnAttachedStateChanged(bool attached) { 103 void ServiceWorkerDevToolsAgentHost::OnAttachedStateChanged(bool attached) {
101 BrowserThread::PostTask(BrowserThread::IO, FROM_HERE, 104 BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
102 base::Bind(&SetDevToolsAttachedOnIO, 105 base::Bind(&SetDevToolsAttachedOnIO,
103 service_worker_->context_weak(), 106 service_worker_->context_weak(),
104 service_worker_->version_id(), 107 service_worker_->version_id(),
105 attached)); 108 attached));
106 } 109 }
107 110
111 void ServiceWorkerDevToolsAgentHost::WorkerVersionInstalled() {
112 version_installed_time_ = base::Time::Now();
113 }
114
115 void ServiceWorkerDevToolsAgentHost::WorkerVersionDoomed() {
116 version_doomed_time_ = base::Time::Now();
117 }
118
108 int64_t ServiceWorkerDevToolsAgentHost::service_worker_version_id() const { 119 int64_t ServiceWorkerDevToolsAgentHost::service_worker_version_id() const {
109 return service_worker_->version_id(); 120 return service_worker_->version_id();
110 } 121 }
111 122
112 GURL ServiceWorkerDevToolsAgentHost::scope() const { 123 GURL ServiceWorkerDevToolsAgentHost::scope() const {
113 return service_worker_->scope(); 124 return service_worker_->scope();
114 } 125 }
115 126
116 bool ServiceWorkerDevToolsAgentHost::Matches( 127 bool ServiceWorkerDevToolsAgentHost::Matches(
117 const ServiceWorkerIdentifier& other) { 128 const ServiceWorkerIdentifier& other) {
118 return service_worker_->Matches(other); 129 return service_worker_->Matches(other);
119 } 130 }
120 131
121 ServiceWorkerDevToolsAgentHost::~ServiceWorkerDevToolsAgentHost() { 132 ServiceWorkerDevToolsAgentHost::~ServiceWorkerDevToolsAgentHost() {
122 ServiceWorkerDevToolsManager::GetInstance()->RemoveInspectedWorkerData( 133 ServiceWorkerDevToolsManager::GetInstance()->RemoveInspectedWorkerData(
123 worker_id()); 134 worker_id());
124 } 135 }
125 136
126 } // namespace content 137 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698