[ServiceWorker] Set blob_size of ServiceWorkerFetchRequest as kuint64max when it is unknown.

[ServiceWorker] Set blob_size of ServiceWorkerFetchRequest as kuint64max when it is unknown.

This patch and https://codereview.chromium.org/514113003 will fix the wrong size of blob which can be get by calling FetchEvent.request.body.asBody() of the fetch event which body includes a file item.


BUG=402387
Committed: https://crrev.com/792dcd9ba6050f1dce72f9a7e1961e0f0ceabc2c
Cr-Commit-Position: refs/heads/master@{#292566}

[horo, 2014-08-28 12:02:25]

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

[horo, 2014-08-28 12:02:25]

michaeln@
Could you please review?

[michaeln, 2014-08-28 22:46:35]

lgtm

https://codereview.chromium.org/517543002/diff/1/content/browser/service_work...
File content/browser/service_worker/service_worker_url_request_job.cc (right):

https://codereview.chromium.org/517543002/diff/1/content/browser/service_work...
content/browser/service_worker/service_worker_url_request_job.cc:270: return
false;
nit: maybe a blank line here

https://codereview.chromium.org/517543002/diff/1/content/browser/service_work...
content/browser/service_worker/service_worker_url_request_job.cc:287: }
nit: and here

[horo, 2014-08-29 01:13:04]

Thank you!

https://codereview.chromium.org/517543002/diff/1/content/browser/service_work...
File content/browser/service_worker/service_worker_url_request_job.cc (right):

https://codereview.chromium.org/517543002/diff/1/content/browser/service_work...
content/browser/service_worker/service_worker_url_request_job.cc:270: return
false;
On 2014/08/28 22:46:34, michaeln wrote:
> nit: maybe a blank line here

Done.

https://codereview.chromium.org/517543002/diff/1/content/browser/service_work...
content/browser/service_worker/service_worker_url_request_job.cc:287: }
On 2014/08/28 22:46:34, michaeln wrote:
> nit: and here

Done.

[horo, 2014-08-29 02:26:32]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-08-29 02:27:01]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/horo@chromium.org/517543002/20001

[commit-bot: I haz the power, 2014-08-29 03:43:17]

Committed patchset #2 (id:20001) as 66febf39d3527bbee0de1a3f9daeb27c3a1e9f5d

[commit-bot: I haz the power, 2014-09-10 03:06:05]

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