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

Unified Diff: content/browser/service_worker/service_worker_url_request_job.h

Issue 1146253004: Add workerReady timing for ServiceWorker controlled requests [2/3] (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/service_worker/service_worker_url_request_job.h
diff --git a/content/browser/service_worker/service_worker_url_request_job.h b/content/browser/service_worker/service_worker_url_request_job.h
index 96e500728e4d55971d6d810827ca0fd6349ce320..18afaa7d900cfc8a83ec0878d3aff82353020b23 100644
--- a/content/browser/service_worker/service_worker_url_request_job.h
+++ b/content/browser/service_worker/service_worker_url_request_job.h
@@ -43,6 +43,7 @@ class ServiceWorkerFetchDispatcher;
class ServiceWorkerProviderHost;
class ServiceWorkerVersion;
class Stream;
+struct ResourceResponseInfo;
class CONTENT_EXPORT ServiceWorkerURLRequestJob
: public net::URLRequestJob,
@@ -109,16 +110,14 @@ class CONTENT_EXPORT ServiceWorkerURLRequestJob
// StreamRegisterObserver override:
void OnStreamRegistered(Stream* stream) override;
- void GetExtraResponseInfo(
- bool* was_fetched_via_service_worker,
- bool* was_fallback_required_by_service_worker,
- GURL* original_url_via_service_worker,
- blink::WebServiceWorkerResponseType* response_type_via_service_worker,
- base::TimeTicks* worker_start_time) const;
+ void GetExtraResponseInfo(ResourceResponseInfo* response_info) const;
const base::TimeTicks& worker_start_time() const {
return worker_start_time_;
}
+ const base::TimeTicks& worker_ready_time() const {
+ return worker_ready_time_;
+ }
protected:
~ServiceWorkerURLRequestJob() override;
@@ -172,6 +171,7 @@ class CONTENT_EXPORT ServiceWorkerURLRequestJob
// Timing info to show on the popup in Devtools' Network tab.
net::LoadTimingInfo load_timing_info_;
base::TimeTicks worker_start_time_;
+ base::TimeTicks worker_ready_time_;
base::Time response_time_;
ResponseType response_type_;

Powered by Google App Engine
This is Rietveld 408576698