[ServiceWorker] Treat the rejected result of onFetch event as a Network Error.

[ServiceWorker] Treat the rejected result of onFetch event as a Network Error.

In current code the rejected result of onFetch event is treated as 500 respose.
But it should be treated as a Network Error.

Step1: [ServiceWorker] Remove the test for the rejected result of FetchEvent from fetch-event.html
  https://codereview.chromium.org/588993002/
Step2: this patch
  [ServiceWorker] Treat the rejected result of onFetch as a Network Error.
  https://codereview.chromium.org/590043002/
Step3: [ServiceWorker] Add test for the rejected result of FetchEvent using XHR.
  https://codereview.chromium.org/588093003/

BUG=411173
Committed: https://crrev.com/8faf94721b6ec90f92404234f0c227e2e501c102
Cr-Commit-Position: refs/heads/master@{#296080}

[horo, 2014-09-22 05:27:21]

Patchset #1 (id:1) has been deleted

[horo, 2014-09-22 05:29:33]

horo@chromium.org changed reviewers:
+ yhirano@chromium.org

[horo, 2014-09-22 05:29:33]

yhirano@
Could you please review?

[yhirano, 2014-09-22 07:42:35]

lgtm

https://codereview.chromium.org/590043002/diff/20001/content/browser/service_...
File content/browser/service_worker/service_worker_url_request_job.cc (right):

https://codereview.chromium.org/590043002/diff/20001/content/browser/service_...
content/browser/service_worker/service_worker_url_request_job.cc:384: // Treat a
response whose status is 0 as an error.
mentioning "Network Error" makes more sense because "error" word is also used in
DeliverErrorResponse.

[horo, 2014-09-22 08:38:20]

https://codereview.chromium.org/590043002/diff/20001/content/browser/service_...
File content/browser/service_worker/service_worker_url_request_job.cc (right):

https://codereview.chromium.org/590043002/diff/20001/content/browser/service_...
content/browser/service_worker/service_worker_url_request_job.cc:384: // Treat a
response whose status is 0 as an error.
On 2014/09/22 07:42:35, yhirano wrote:
> mentioning "Network Error" makes more sense because "error" word is also used
in
> DeliverErrorResponse.

Done.

[yhirano, 2014-09-22 08:41:59]

On 2014/09/22 08:38:20, horo wrote:
>
https://codereview.chromium.org/590043002/diff/20001/content/browser/service_...
> File content/browser/service_worker/service_worker_url_request_job.cc (right):
> 
>
https://codereview.chromium.org/590043002/diff/20001/content/browser/service_...
> content/browser/service_worker/service_worker_url_request_job.cc:384: // Treat
a
> response whose status is 0 as an error.
> On 2014/09/22 07:42:35, yhirano wrote:
> > mentioning "Network Error" makes more sense because "error" word is also
used
> in
> > DeliverErrorResponse.
> 
> Done.

lgtm again

[horo, 2014-09-22 21:36:48]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-09-22 21:37:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/590043002/40001

[commit-bot: I haz the power, 2014-09-22 22:57:28]

Committed patchset #2 (id:40001) as 79369b47e475ebb40785eca8bdc269b9c543ebf3

[commit-bot: I haz the power, 2014-09-22 22:57:58]

Patchset 2 (id:??) landed as
https://crrev.com/8faf94721b6ec90f92404234f0c227e2e501c102
Cr-Commit-Position: refs/heads/master@{#296080}