Set service worker response info data for foreign fetch intercepted fetches.

content/browser/loader/resource_loader.cc

Index: content/browser/loader/resource_loader.cc
diff --git a/content/browser/loader/resource_loader.cc b/content/browser/loader/resource_loader.cc
index 40e1f2a60746f96da3530f967c37a3dd82ee3edf..ca8d7b75f4a2f2c26bc0baec629bce6d12a19115 100644
--- a/content/browser/loader/resource_loader.cc
+++ b/content/browser/loader/resource_loader.cc
@@ -21,6 +21,7 @@
 #include "content/browser/loader/resource_loader_delegate.h"
 #include "content/browser/loader/resource_request_info_impl.h"
 #include "content/browser/service_worker/service_worker_request_handler.h"
+#include "content/browser/service_worker/service_worker_response_info.h"
 #include "content/browser/ssl/ssl_client_auth_handler.h"
 #include "content/browser/ssl/ssl_manager.h"
 #include "content/browser/ssl/ssl_policy.h"
@@ -103,9 +104,9 @@ void PopulateResourceResponse(ResourceRequestInfoImpl* info,
       content::ResourceRequestInfo::ForRequest(request);
   if (request_info)
     response->head.is_using_lofi = request_info->IsUsingLoFi();
-  if (ServiceWorkerRequestHandler* handler =
-          ServiceWorkerRequestHandler::GetHandler(request)) {
-    handler->GetExtraResponseInfo(&response->head);
+  if (ServiceWorkerResponseInfo* service_worker_info =

[mmenke, 2016/04/21 18:30:31]

Please move declaration of ServiceWorkerResponseInfo outside of the if, while
you're here.  Declaring variables in if's is just not common in chrome.  I'd
suggest moving the initialization outside the if as well, just because that's
how things are usually done.

[Marijn Kruisselbrink, 2016/04/21 19:20:45]

Done

+          ServiceWorkerResponseInfo::ForRequest(request)) {
+    service_worker_info->GetExtraResponseInfo(&response->head);
   }
   AppCacheInterceptor::GetExtraResponseInfo(
       request, &response->head.appcache_id,