Skip logging spurious errors for ServiceWorker URLRequests that were spawned

extensions/browser/extension_protocols.cc

Index: extensions/browser/extension_protocols.cc
diff --git a/extensions/browser/extension_protocols.cc b/extensions/browser/extension_protocols.cc
index 1fcabe19bb2119b679e9e0689c3ff3721f44a9a2..f6f5737ef56858c01a1e306a30efeef8c5335c8f 100644
--- a/extensions/browser/extension_protocols.cc
+++ b/extensions/browser/extension_protocols.cc
@@ -329,9 +329,17 @@ bool AllowExtensionResourceLoad(net::URLRequest* request,
 
   // We have seen crashes where info is NULL: crbug.com/52374.
   if (!info) {
-    LOG(ERROR) << "Allowing load of " << request->url().spec()
-               << "from unknown origin. Could not find user data for "
-               << "request.";
+    // SeviceWorker net requests created through ServiceWorkerWriteToCacheJob
+    // does not have ResourceRequestInfo associated with them. So skip logging

[asargent_no_longer_on_chrome, 2016/03/15 16:41:29]

grammar nit: should be "... do not have..."

[lazyboy, 2016/03/16 19:48:25]

Done.

+    // spurious errors below.
+    // TODO(falken): Either consider attaching ResourceRequestInfo to these or
+    // finish refactoring ServiceWorkerWriteToCacheJob so that it doesn't spawn
+    // a new URLRequest.
+    if (!ResourceRequestInfo::OriginatedFromServiceWorker(request)) {
+      LOG(ERROR) << "Allowing load of " << request->url().spec()
+                 << "from unknown origin. Could not find user data for "
+                 << "request.";
+    }
     return true;
   }