Revert of [CacheStorage] Clean up CacheStorageCache lifetimes

Revert of [CacheStorage] Clean up CacheStorageCache lifetimes (patchset #3 id:60001 of https://codereview.chromium.org/2040173002/ )

Reason for revert:
Rather than carefully control the lifetimes of the refptrs, I'm going to give explicit ownership of caches to CacheStorage so that on ~CacheStorage the caches die. CacheStorage will give clients handles to the caches (that extend the lifetime but not past destruction).

Original issue's description:
> [CacheStorage] Clean up CacheStorageCache lifetimes
>
> CacheStorageCache refptrs are held in callbacks to cache operations, meaning that they won't be destructed
> on profile shutdown.  This CL moves the refptrs to member variables so that they will.
>
> BUG=617683
>
> Committed: https://crrev.com/f9b13e0a9790f868a54ad4f3d1268d7369239ed4
> Cr-Commit-Position: refs/heads/master@{#398529}

TBR=jochen@chromium.org,nhiroki@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=617683
Committed: https://crrev.com/ec7a87d4b4e2707ab7ad19a3125dbaed9cd2ad2a
Cr-Commit-Position: refs/heads/master@{#398873}

[jkarlin, 2016-06-09 13:44:36]

The CQ bit was checked by jkarlin@chromium.org

[jkarlin, 2016-06-09 13:44:37]

Created Revert of [CacheStorage] Clean up CacheStorageCache lifetimes

[commit-bot: I haz the power, 2016-06-09 13:44:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2040373006/1

[commit-bot: I haz the power, 2016-06-09 14:33:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-09 14:33:26]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-09 14:35:19]

Description was changed from

==========
Revert of [CacheStorage] Clean up CacheStorageCache lifetimes (patchset #3
id:60001 of https://codereview.chromium.org/2040173002/ )

Reason for revert:
Rather than carefully control the lifetimes of the refptrs, I'm going to give
explicit ownership of caches to CacheStorage so that on ~CacheStorage the caches
die. CacheStorage will give clients handles to the caches (that extend the
lifetime but not past destruction).

Original issue's description:
> [CacheStorage] Clean up CacheStorageCache lifetimes
> 
> CacheStorageCache refptrs are held in callbacks to cache operations, meaning
that they won't be destructed
> on profile shutdown.  This CL moves the refptrs to member variables so that
they will.
> 
> BUG=617683
> 
> Committed: https://crrev.com/f9b13e0a9790f868a54ad4f3d1268d7369239ed4
> Cr-Commit-Position: refs/heads/master@{#398529}

TBR=jochen@chromium.org,nhiroki@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=617683
==========

to

==========
Revert of [CacheStorage] Clean up CacheStorageCache lifetimes (patchset #3
id:60001 of https://codereview.chromium.org/2040173002/ )

Reason for revert:
Rather than carefully control the lifetimes of the refptrs, I'm going to give
explicit ownership of caches to CacheStorage so that on ~CacheStorage the caches
die. CacheStorage will give clients handles to the caches (that extend the
lifetime but not past destruction).

Original issue's description:
> [CacheStorage] Clean up CacheStorageCache lifetimes
>
> CacheStorageCache refptrs are held in callbacks to cache operations, meaning
that they won't be destructed
> on profile shutdown.  This CL moves the refptrs to member variables so that
they will.
>
> BUG=617683
>
> Committed: https://crrev.com/f9b13e0a9790f868a54ad4f3d1268d7369239ed4
> Cr-Commit-Position: refs/heads/master@{#398529}

TBR=jochen@chromium.org,nhiroki@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=617683

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

[commit-bot: I haz the power, 2016-06-09 14:35:20]

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