ServiceWorker browser test fix

Fix a browser tests to immediately grab a reference to the a response blob instead of deferring it.

BUG=381827
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=275624

[michaeln, 2014-06-06 22:55:39]

ptal

i had started out trying to have the version grab the version immedately upon
receipt of the ipc (which is a good idea), but i didn't want to have sw system
addref the blobcontext, so it wasn't working out like that
https://codereview.chromium.org/320553003/

this browsertest bug can be fixed more easily (this cl), maybe i'll pick up that
other change some other time

[michaeln, 2014-06-06 22:56:19]

errr... "grab the reference immedately"

[michaeln, 2014-06-06 22:59:24]

https://codereview.chromium.org/321613002/diff/40001/content/browser/service_...
File content/browser/service_worker/service_worker_browsertest.cc (right):

https://codereview.chromium.org/321613002/diff/40001/content/browser/service_...
content/browser/service_worker/service_worker_browsertest.cc:316: FetchResult
fetch_result;
ooops, this isn't right yet

[michaeln, 2014-06-06 23:31:56]

ok, should be fixed now

[falken, 2014-06-06 23:55:50]

lgtm with a suggestion

https://codereview.chromium.org/321613002/diff/60001/content/browser/service_...
File content/browser/service_worker/service_worker_browsertest.cc (right):

https://codereview.chromium.org/321613002/diff/60001/content/browser/service_...
content/browser/service_worker/service_worker_browsertest.cc:107:
webkit_blob::BlobDataHandle* blob_data_handle) {
Would it be useful to add ASSERT_TRUE(blob_data_handle) ? It would have made the
bug this CL fixes more obvious.

[michaeln, 2014-06-07 00:00:02]

https://codereview.chromium.org/321613002/diff/60001/content/browser/service_...
File content/browser/service_worker/service_worker_browsertest.cc (right):

https://codereview.chromium.org/321613002/diff/60001/content/browser/service_...
content/browser/service_worker/service_worker_browsertest.cc:107:
webkit_blob::BlobDataHandle* blob_data_handle) {
On 2014/06/06 23:55:50, falken wrote:
> Would it be useful to add ASSERT_TRUE(blob_data_handle) ? It would have made
the
> bug this CL fixes more obvious.

Done

[michaeln, 2014-06-07 00:03:02]

The CQ bit was checked by michaeln@chromium.org

[commit-bot: I haz the power, 2014-06-07 00:04:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/michaeln@chromium.org/321613002/100001

[commit-bot: I haz the power, 2014-06-07 05:20:53]

Change committed as 275624