CacheStorage: Merge CacheStorageListener into CacheStorageDispatcherHost (1)

CacheStorage: Merge CacheStorageListener into CacheStorageDispatcherHost (1)

CacheStorageListener and CacheStorageDispatcher are now in a 1:1 relationship
and the indirection is no longer necessary. As a first step of merge, this CL
moves message handlers for CacheStorage API from the listener to the dispatcher.

BUG=474233
TEST=content_unittests --gtest_filter=CacheStorage*

Committed: https://crrev.com/34cbfce4cae255c11fb19059ea4a31166768cdcf
Cr-Commit-Position: refs/heads/master@{#327199}

[nhiroki, 2015-04-27 05:37:31]

Patchset #1 (id:1) has been deleted

[nhiroki, 2015-04-27 05:37:36]

Patchset #1 (id:20001) has been deleted

[nhiroki, 2015-04-27 05:37:43]

Patchset #1 (id:40001) has been deleted

[nhiroki, 2015-04-27 05:37:49]

Patchset #1 (id:60001) has been deleted

[nhiroki, 2015-04-27 05:37:54]

Patchset #1 (id:80001) has been deleted

[nhiroki, 2015-04-27 05:52:12]

Patchset #1 (id:100001) has been deleted

[nhiroki, 2015-04-27 06:06:31]

nhiroki@chromium.org changed reviewers:
+ jsbell@chromium.org

[nhiroki, 2015-04-27 06:06:31]

Hi, can you review this? Thanks.

[jsbell, 2015-04-27 23:15:39]

lgtm

Thanks for picking this up! Normally we'd want TODOs explaining why there is
temporary duplicated code, but I'm sure the follow-up CL will be equally simple
and land soon too.

[nhiroki, 2015-04-27 23:42:19]

On 2015/04/27 23:15:39, jsbell wrote:
> lgtm
> 
> Thanks for picking this up! Normally we'd want TODOs explaining why there is
> temporary duplicated code, but I'm sure the follow-up CL will be equally
simple
> and land soon too.

Thank you for reviewing. Added such TODO comment, just in case :)

[nhiroki, 2015-04-27 23:42:22]

The CQ bit was checked by nhiroki@chromium.org

[nhiroki, 2015-04-27 23:42:22]

The patchset sent to the CQ was uploaded after l-g-t-m from jsbell@chromium.org
Link to the patchset: https://codereview.chromium.org/1101733005/#ps140001
(title: "add TODO comment")

[commit-bot: I haz the power, 2015-04-27 23:43:25]

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

[commit-bot: I haz the power, 2015-04-28 00:50:29]

Committed patchset #2 (id:140001)

[commit-bot: I haz the power, 2015-04-28 00:52:04]

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

[pdr., 2015-04-28 04:00:31]

On 2015/04/28 at 00:52:04, commit-bot wrote:
> Patchset 2 (id:??) landed as
https://crrev.com/34cbfce4cae255c11fb19059ea4a31166768cdcf
> Cr-Commit-Position: refs/heads/master@{#327199}

I think this patch caused layouttest failures on the Blink tree, see
crbug.com/481784 for details.

[pdr., 2015-04-28 04:00:42]

A revert of this CL (patchset #2 id:140001) has been created in
https://codereview.chromium.org/1111703002/ by pdr@chromium.org.

The reason for reverting is: I think this patch caused layouttest failures on
the Blink tree, see crbug.com/481784 for details..