[CacheStorage] Don't store open caches in Blink

[CacheStorage] Don't store open caches in Blink

Blink objects were storing cache objects so that they could be returned quickly
on subsequent open calls. This is problematic as there can be many contexts
(different windows, workers, service workers, and dev tools) opening/deleting
caches. If one of them deletes the cache then all of the others need to be
invalidated. Rather than synchronize the various contexts, for now just remove
the stored caches from Blink.

BUG=639034
Committed: https://crrev.com/db792ded02006fe0e9b4230c1b904c0560302007
Cr-Commit-Position: refs/heads/master@{#414147}

[jkarlin, 2016-08-24 19:02:46]

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

[jkarlin, 2016-08-24 19:03:44]

jkarlin@chromium.org changed reviewers:
+ jochen@chromium.org, jsbell@chromium.org

[jkarlin, 2016-08-24 19:03:45]

jsbell@ PTAL at everything

jochen@ PTAL for owners

Thanks!

[commit-bot: I haz the power, 2016-08-24 19:04:04]

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

[jkarlin, 2016-08-24 19:04:18]

Description was changed from

==========
[CacheStorage] Don't store open caches in Blink

Blink objects were storing cache objects so that they could be returned quickly
on subsequent open calls. This is problematic as there can be many contexts
(different windows, workers, service workers, and dev tools) opening/deleting
caches. If one of them deletes the cache then all of the others need to be
invalidated. Rather than synchronize the various contexts, for now just remove
the stored caches from Blink.

BUG=639034
==========

to

==========
[CacheStorage] Don't store open caches in Blink

Blink objects were storing cache objects so that they could be returned quickly
on subsequent open calls. This is problematic as there can be many contexts
(different windows, workers, service workers, and dev tools) opening/deleting
caches. If one of them deletes the cache then all of the others need to be
invalidated. Rather than synchronize the various contexts, for now just remove
the stored caches from Blink.

BUG=639034
==========

[jkarlin, 2016-08-24 19:04:18]

jkarlin@chromium.org changed reviewers:
- jochen@chromium.org

[jkarlin, 2016-08-24 19:04:37]

-r jochen as jsbell@ is an owner of the directory in question.

[jsbell, 2016-08-24 19:24:35]

lgtm

[jkarlin, 2016-08-24 21:05:58]

The CQ bit was unchecked by jkarlin@chromium.org

[jkarlin, 2016-08-24 21:06:07]

The CQ bit was checked by jkarlin@chromium.org

[commit-bot: I haz the power, 2016-08-24 21:06:27]

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-24 21:15:58]

Description was changed from

==========
[CacheStorage] Don't store open caches in Blink

Blink objects were storing cache objects so that they could be returned quickly
on subsequent open calls. This is problematic as there can be many contexts
(different windows, workers, service workers, and dev tools) opening/deleting
caches. If one of them deletes the cache then all of the others need to be
invalidated. Rather than synchronize the various contexts, for now just remove
the stored caches from Blink.

BUG=639034
==========

to

==========
[CacheStorage] Don't store open caches in Blink

Blink objects were storing cache objects so that they could be returned quickly
on subsequent open calls. This is problematic as there can be many contexts
(different windows, workers, service workers, and dev tools) opening/deleting
caches. If one of them deletes the cache then all of the others need to be
invalidated. Rather than synchronize the various contexts, for now just remove
the stored caches from Blink.

BUG=639034
==========

[commit-bot: I haz the power, 2016-08-24 21:15:59]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-24 21:19:51]

Description was changed from

==========
[CacheStorage] Don't store open caches in Blink

Blink objects were storing cache objects so that they could be returned quickly
on subsequent open calls. This is problematic as there can be many contexts
(different windows, workers, service workers, and dev tools) opening/deleting
caches. If one of them deletes the cache then all of the others need to be
invalidated. Rather than synchronize the various contexts, for now just remove
the stored caches from Blink.

BUG=639034
==========

to

==========
[CacheStorage] Don't store open caches in Blink

Blink objects were storing cache objects so that they could be returned quickly
on subsequent open calls. This is problematic as there can be many contexts
(different windows, workers, service workers, and dev tools) opening/deleting
caches. If one of them deletes the cache then all of the others need to be
invalidated. Rather than synchronize the various contexts, for now just remove
the stored caches from Blink.

BUG=639034

Committed: https://crrev.com/db792ded02006fe0e9b4230c1b904c0560302007
Cr-Commit-Position: refs/heads/master@{#414147}
==========

[commit-bot: I haz the power, 2016-08-24 21:19:52]

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