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

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

Issue 2654663004: [Not for review] record detailed time breakdown of SW related requests.
Patch Set: add stream uma Created 3 years, 10 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_fetch_dispatcher.h
diff --git a/content/browser/service_worker/service_worker_fetch_dispatcher.h b/content/browser/service_worker/service_worker_fetch_dispatcher.h
index 192e17beb009526e1b3f757bec186f576da1c659..3c3ed90c7b6bf8f7a257793ee81fad6ea4cbae02 100644
--- a/content/browser/service_worker/service_worker_fetch_dispatcher.h
+++ b/content/browser/service_worker/service_worker_fetch_dispatcher.h
@@ -61,6 +61,14 @@ class CONTENT_EXPORT ServiceWorkerFetchDispatcher {
// |prepare_callback| as an intermediate step once the version is activated
// and running.
void Run();
+ void set_renderer_request_time(base::TimeTicks t) {
+ renderer_request_time_ = t;
+ }
+ void set_handler_creation_time(base::TimeTicks t) {
+ handler_creation_time_ = t;
+ }
+ void set_job_creation_time(base::TimeTicks t) { job_creation_time_ = t; }
+ void set_worker_start_time(base::TimeTicks t) { worker_start_time_ = t; }
private:
class ResponseCallback;
@@ -75,7 +83,8 @@ class CONTENT_EXPORT ServiceWorkerFetchDispatcher {
void DidFail(ServiceWorkerStatusCode status);
void DidFinish(int request_id,
ServiceWorkerFetchEventResult fetch_result,
- const ServiceWorkerResponse& response);
+ const ServiceWorkerResponse& response,
+ base::TimeTicks response_time);
void Complete(ServiceWorkerStatusCode status,
ServiceWorkerFetchEventResult fetch_result,
const ServiceWorkerResponse& response);
@@ -84,6 +93,12 @@ class CONTENT_EXPORT ServiceWorkerFetchDispatcher {
ServiceWorkerVersion* version,
int event_finish_id,
scoped_refptr<URLLoaderAssets> url_loader_assets,
+ base::TimeTicks renderer_request_time,
+ base::TimeTicks handler_creation_time,
+ base::TimeTicks job_creation_time,
+ base::TimeTicks worker_start_time,
+ base::TimeTicks worker_ready_time,
+ base::TimeTicks dispatch_event_time_ticks,
ServiceWorkerStatusCode status,
base::Time dispatch_event_time);
@@ -102,6 +117,11 @@ class CONTENT_EXPORT ServiceWorkerFetchDispatcher {
mojom::FetchEventPreloadHandlePtr preload_handle_;
+ base::TimeTicks renderer_request_time_;
+ base::TimeTicks handler_creation_time_;
+ base::TimeTicks job_creation_time_;
+ base::TimeTicks worker_start_time_;
+ base::TimeTicks worker_ready_time_;
base::WeakPtrFactory<ServiceWorkerFetchDispatcher> weak_factory_;
DISALLOW_COPY_AND_ASSIGN(ServiceWorkerFetchDispatcher);

Powered by Google App Engine
This is Rietveld 408576698