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

Side by Side Diff: content/browser/service_worker/service_worker_controllee_request_handler.cc

Issue 515753003: Add ServiceWorker timing information on the popup panel in DevTools's Network tab (2/2) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Incorporate the reviews Created 6 years, 3 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/service_worker/service_worker_controllee_request_handl er.h" 5 #include "content/browser/service_worker/service_worker_controllee_request_handl er.h"
6 6
7 #include "content/browser/service_worker/service_worker_context_core.h" 7 #include "content/browser/service_worker/service_worker_context_core.h"
8 #include "content/browser/service_worker/service_worker_metrics.h" 8 #include "content/browser/service_worker/service_worker_metrics.h"
9 #include "content/browser/service_worker/service_worker_provider_host.h" 9 #include "content/browser/service_worker/service_worker_provider_host.h"
10 #include "content/browser/service_worker/service_worker_registration.h" 10 #include "content/browser/service_worker/service_worker_registration.h"
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 // fallback to network. 94 // fallback to network.
95 job_ = NULL; 95 job_ = NULL;
96 return NULL; 96 return NULL;
97 } 97 }
98 98
99 return job_.get(); 99 return job_.get();
100 } 100 }
101 101
102 void ServiceWorkerControlleeRequestHandler::GetExtraResponseInfo( 102 void ServiceWorkerControlleeRequestHandler::GetExtraResponseInfo(
103 bool* was_fetched_via_service_worker, 103 bool* was_fetched_via_service_worker,
104 GURL* original_url_via_service_worker) const { 104 GURL* original_url_via_service_worker,
105 base::TimeTicks* fetch_start_time,
106 base::TimeTicks* fetch_ready_time,
107 base::TimeTicks* fetch_end_time) const {
105 if (!job_.get()) { 108 if (!job_.get()) {
106 *was_fetched_via_service_worker = false; 109 *was_fetched_via_service_worker = false;
107 *original_url_via_service_worker = GURL(); 110 *original_url_via_service_worker = GURL();
108 return; 111 return;
109 } 112 }
110 job_->GetExtraResponseInfo(was_fetched_via_service_worker, 113 job_->GetExtraResponseInfo(was_fetched_via_service_worker,
111 original_url_via_service_worker); 114 original_url_via_service_worker,
115 fetch_start_time,
116 fetch_ready_time,
117 fetch_end_time);
112 } 118 }
113 119
114 void ServiceWorkerControlleeRequestHandler::PrepareForMainResource( 120 void ServiceWorkerControlleeRequestHandler::PrepareForMainResource(
115 const GURL& url) { 121 const GURL& url) {
116 DCHECK(job_.get()); 122 DCHECK(job_.get());
117 DCHECK(context_); 123 DCHECK(context_);
118 // The corresponding provider_host may already have associated a registration 124 // The corresponding provider_host may already have associated a registration
119 // in redirect case, unassociate it now. 125 // in redirect case, unassociate it now.
120 provider_host_->UnassociateRegistration(); 126 provider_host_->UnassociateRegistration();
121 127
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
185 } 191 }
186 192
187 void ServiceWorkerControlleeRequestHandler::PrepareForSubResource() { 193 void ServiceWorkerControlleeRequestHandler::PrepareForSubResource() {
188 DCHECK(job_.get()); 194 DCHECK(job_.get());
189 DCHECK(context_); 195 DCHECK(context_);
190 DCHECK(provider_host_->active_version()); 196 DCHECK(provider_host_->active_version());
191 job_->ForwardToServiceWorker(); 197 job_->ForwardToServiceWorker();
192 } 198 }
193 199
194 } // namespace content 200 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698