ServiceWorker: Change worker script fetch error code(2/3).

content/browser/service_worker/service_worker_write_to_cache_job.cc

Index: content/browser/service_worker/service_worker_write_to_cache_job.cc
diff --git a/content/browser/service_worker/service_worker_write_to_cache_job.cc b/content/browser/service_worker/service_worker_write_to_cache_job.cc
index bb923eb013aa331f52594f6aa74682249859329d..e6cfb51f4b0b476bbf31dceb900d2afb9cafe9df 100644
--- a/content/browser/service_worker/service_worker_write_to_cache_job.cc
+++ b/content/browser/service_worker/service_worker_write_to_cache_job.cc
@@ -280,6 +280,7 @@ void ServiceWorkerWriteToCacheJob::OnReceivedRedirect(
   // Script resources can't redirect.
   AsyncNotifyDoneHelper(net::URLRequestStatus(
       net::URLRequestStatus::FAILED, net::ERR_FAILED));
+  version_->NotifyStartWorkerURLRequestError(SERVICE_WORKER_ERROR_SECURITY);

[michaeln, 2014/09/13 01:21:34]

This class is used for more than the main resource, its used for imports as
well. This class deals with a single resource load, the starting of a worker is
more involved than a single resource load. I don't this class can/should make
this call.

[xiang, 2014/09/16 06:24:31]

I see. However from the spec seems the imports redirect should be permitted, I
changed the code to only fail when load main script redirected.

 }
 
 void ServiceWorkerWriteToCacheJob::OnAuthRequired(
@@ -337,6 +338,7 @@ void ServiceWorkerWriteToCacheJob::OnResponseStarted(
   if (request->GetResponseCode() / 100 != 2) {
     AsyncNotifyDoneHelper(net::URLRequestStatus(
         net::URLRequestStatus::FAILED, net::ERR_FAILED));
+    version_->NotifyStartWorkerURLRequestError(SERVICE_WORKER_ERROR_NETWORK);
     // TODO(michaeln): Instead of error'ing immediately, send the net
     // response to our consumer, just don't cache it?
     return;
@@ -350,6 +352,7 @@ void ServiceWorkerWriteToCacheJob::OnResponseStarted(
         mime_type != "application/javascript") {
       AsyncNotifyDoneHelper(net::URLRequestStatus(
           net::URLRequestStatus::FAILED, net::ERR_FAILED));
+      version_->NotifyStartWorkerURLRequestError(SERVICE_WORKER_ERROR_SECURITY);
       return;
     }
   }