service worker: Clean up fetch event if URLRequest gets cancelled.

service worker: Clean up the fetch event request if URLRequest gets cancelled.

Before this patch, an inflight fetch event would never be
finished, since the request being cancelled results in the
URLRequestJob being destroyed, so the the response callback
is never called. Eventually the event would timeout, but this
is an error state that terminates the worker. It's better to
listen for the response and finish the request cleanly.

BUG=616331
Committed: https://crrev.com/9ed9574ae81390911dd582a0d5a49a4c0430eafa
Cr-Commit-Position: refs/heads/master@{#403632}

[falken, 2016-07-04 02:20:34]

Description was changed from

==========
Clean up fetch event if URLRequest gets cancelled.

BUG=
==========

to

==========
service worker: Clean up the fetch event request if URLRequest gets cancelled.

BUG=616331
==========

[falken, 2016-07-04 02:21:32]

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

[falken, 2016-07-04 02:21:33]

horo@: PTAL

[falken, 2016-07-04 02:24:47]

Description was changed from

==========
service worker: Clean up the fetch event request if URLRequest gets cancelled.

BUG=616331
==========

to

==========
service worker: Clean up the fetch event request if URLRequest gets cancelled.

Before this patch, an inflight fetch event would never be
finished, since the request being cancelled results in the
URLRequestJob being destroyed, so the the response callback
is never called. Eventually the event would timeout, but this
is an error state that terminates the worker. It's better to
listen for the response and finish the request cleanly.

BUG=616331
==========

[horo, 2016-07-04 03:19:34]

lgtm

[falken, 2016-07-04 03:21:40]

The CQ bit was checked by falken@chromium.org

[commit-bot: I haz the power, 2016-07-04 03:21:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-04 03:52:19]

Description was changed from

==========
service worker: Clean up the fetch event request if URLRequest gets cancelled.

Before this patch, an inflight fetch event would never be
finished, since the request being cancelled results in the
URLRequestJob being destroyed, so the the response callback
is never called. Eventually the event would timeout, but this
is an error state that terminates the worker. It's better to
listen for the response and finish the request cleanly.

BUG=616331
==========

to

==========
service worker: Clean up the fetch event request if URLRequest gets cancelled.

Before this patch, an inflight fetch event would never be
finished, since the request being cancelled results in the
URLRequestJob being destroyed, so the the response callback
is never called. Eventually the event would timeout, but this
is an error state that terminates the worker. It's better to
listen for the response and finish the request cleanly.

BUG=616331
==========

[commit-bot: I haz the power, 2016-07-04 03:52:20]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-04 03:53:58]

Description was changed from

==========
service worker: Clean up the fetch event request if URLRequest gets cancelled.

Before this patch, an inflight fetch event would never be
finished, since the request being cancelled results in the
URLRequestJob being destroyed, so the the response callback
is never called. Eventually the event would timeout, but this
is an error state that terminates the worker. It's better to
listen for the response and finish the request cleanly.

BUG=616331
==========

to

==========
service worker: Clean up the fetch event request if URLRequest gets cancelled.

Before this patch, an inflight fetch event would never be
finished, since the request being cancelled results in the
URLRequestJob being destroyed, so the the response callback
is never called. Eventually the event would timeout, but this
is an error state that terminates the worker. It's better to
listen for the response and finish the request cleanly.

BUG=616331

Committed: https://crrev.com/9ed9574ae81390911dd582a0d5a49a4c0430eafa
Cr-Commit-Position: refs/heads/master@{#403632}
==========

[commit-bot: I haz the power, 2016-07-04 03:54:00]

Patchset 3 (id:??) landed as
https://crrev.com/9ed9574ae81390911dd582a0d5a49a4c0430eafa
Cr-Commit-Position: refs/heads/master@{#403632}