Add workerReady timing for ServiceWorker controlled requests [2/3]

content/browser/service_worker/service_worker_context_request_handler.h

Index: content/browser/service_worker/service_worker_context_request_handler.h
diff --git a/content/browser/service_worker/service_worker_context_request_handler.h b/content/browser/service_worker/service_worker_context_request_handler.h
index e31e1da6cdd2917c66e705a63f9f2eba44d5d829..85db8ebf529499b135615b8aea05a86ed1b51844 100644
--- a/content/browser/service_worker/service_worker_context_request_handler.h
+++ b/content/browser/service_worker/service_worker_context_request_handler.h
@@ -29,12 +29,7 @@ class CONTENT_EXPORT ServiceWorkerContextRequestHandler
       net::NetworkDelegate* network_delegate,
       ResourceContext* resource_context) 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 override;
+  void GetExtraResponseInfo(ResourceResponseInfo* response_info) const override;

[mmenke, 2015/06/08 02:08:42]

Should forward declare ResourceResponseInfo.

[Kunihiko Sakamoto, 2015/06/08 02:25:57]

Since this is an overridden function, I think it's ok to put forward-declaration
only in the base class's header file.

 
  private:
   bool ShouldAddToScriptCache(const GURL& url);