service worker: Refactor blob reading out of ServiceWorkerURLRequestJob

service worker: Refactor blob reading out of ServiceWorkerURLRequestJob

ServiceWorkerURLRequestJob has two distinct states when there is a response
from the service worker: it can be reading a blob or a stream.  To clarify the
states, factor out blob and stream response reading into dedicated classes.

This should also help netlog fit better with the superclass's "request
start" vs "reading response" phases.

Committed: https://crrev.com/92aa26d130fe33ef70c2f1279ffaf042c3618296
Cr-Commit-Position: refs/heads/master@{#415293}

[falken, 2016-08-30 07:47:47]

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

[falken, 2016-08-30 07:47:48]

PTAL.

The final result is to split stream and blob reading into separate classes,
which will look like patch 2290123002.

[horo, 2016-08-30 09:32:17]

lgtm

[falken, 2016-08-30 13:01:01]

The CQ bit was checked by falken@chromium.org

[commit-bot: I haz the power, 2016-08-30 13:01:16]

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-08-30 14:48:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 14:49:52]

Description was changed from

==========
service worker: Refactor blob reading out of ServiceWorkerURLRequestJob

ServiceWorkerURLRequestJob has two distinct states when there is a response
from the service worker: it can be reading a blob or a stream.  To clarify the
states, factor out blob and stream response reading into dedicated classes.

This should also help netlog fit better with the superclass's "request
start" vs "reading response" phases.
==========

to

==========
service worker: Refactor blob reading out of ServiceWorkerURLRequestJob

ServiceWorkerURLRequestJob has two distinct states when there is a response
from the service worker: it can be reading a blob or a stream.  To clarify the
states, factor out blob and stream response reading into dedicated classes.

This should also help netlog fit better with the superclass's "request
start" vs "reading response" phases.

Committed: https://crrev.com/92aa26d130fe33ef70c2f1279ffaf042c3618296
Cr-Commit-Position: refs/heads/master@{#415293}
==========

[commit-bot: I haz the power, 2016-08-30 14:49:53]

Patchset 1 (id:??) landed as
https://crrev.com/92aa26d130fe33ef70c2f1279ffaf042c3618296
Cr-Commit-Position: refs/heads/master@{#415293}