Clear BlobReader in SW browser_tests on the right thread

Clear BlobReader in SW browser_tests on the right thread

BlobReader is created on the IO thread and destroyed on the main thread
on Service Worker tests. However, BlobReader has BlobDataSnapshot and
BlobDataSnapshot has BlobDataItem, which is non-thread safe RefCounted.
That implies BlobReader has to be destroyed on IO thread where
BlobReader was created on.

Review-Url: https://codereview.chromium.org/2681073003
Cr-Commit-Position: refs/heads/master@{#448927}
Committed: https://chromium.googlesource.com/chromium/src/+/a79a95466df9d013721167cf2f0c8ba849214207

[tzik, 2017-02-08 05:53:34]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[tzik, 2017-02-08 05:54:14]

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

[tzik, 2017-02-08 05:59:58]

Description was changed from

==========
Clear BlobReader in SW browser_tests on the right thread
==========

to

==========
Clear BlobReader in SW browser_tests on the right thread

BlobReader is created on the IO thread and destroyed on the main thread
on Service Worker tests. However, BlobReader has BlobDataSnapshot and
BlobDataSnapshot has BlobDataItem, which is non-thread safe RefCounted.
That implies BlobReader has to be destroyed on IO thread where
BlobReader was created on.
==========

[tzik, 2017-02-08 06:00:08]

PTAL

[tzik, 2017-02-08 06:00:13]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-08 06:00:28]

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

[horo, 2017-02-08 06:15:41]

lgtm

[tzik, 2017-02-08 06:27:46]

The CQ bit was unchecked by tzik@chromium.org

[tzik, 2017-02-08 06:27:52]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2017-02-08 06:27:56]

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, 2017-02-08 07:05:59]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486535272643670, "parent_rev":
"b66eb61f525f914b1b478b56e7b4792db7b4c4e5", "commit_rev":
"a79a95466df9d013721167cf2f0c8ba849214207"}

[commit-bot: I haz the power, 2017-02-08 07:06:54]

Description was changed from

==========
Clear BlobReader in SW browser_tests on the right thread

BlobReader is created on the IO thread and destroyed on the main thread
on Service Worker tests. However, BlobReader has BlobDataSnapshot and
BlobDataSnapshot has BlobDataItem, which is non-thread safe RefCounted.
That implies BlobReader has to be destroyed on IO thread where
BlobReader was created on.
==========

to

==========
Clear BlobReader in SW browser_tests on the right thread

BlobReader is created on the IO thread and destroyed on the main thread
on Service Worker tests. However, BlobReader has BlobDataSnapshot and
BlobDataSnapshot has BlobDataItem, which is non-thread safe RefCounted.
That implies BlobReader has to be destroyed on IO thread where
BlobReader was created on.

Review-Url: https://codereview.chromium.org/2681073003
Cr-Commit-Position: refs/heads/master@{#448927}
Committed:
https://chromium.googlesource.com/chromium/src/+/a79a95466df9d013721167cf2f0c...
==========

[commit-bot: I haz the power, 2017-02-08 07:06:56]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/a79a95466df9d013721167cf2f0c...