Fix layering of ApplicationCacheHost

Fix layering of ApplicationCacheHost

This patch moves the frame-walking code from Chromium's WebApplicationCacheHostImpl
into WebCore::ApplicationCacheHost, which breaks that class's dependency on the
public/web API.

That allows blink::WebApplicationCacheHost{,Client} to move to public/platform.
With the addition of a factory method on FrameLoaderClient, this also allos
ApplicationCacheHost.cpp to move next to its header file in core/loader/appcache.

ApplicationCacheHostInternal.h remains for now, but in the long run it can probably
be removed (though the ownership semantics aren't completely clear to me).

BUG=343272
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=167359

[eseidel, 2014-02-14 00:54:31]

FYI, Frame ownership is moving into the embedder:
https://codereview.chromium.org/156123004/

[adamk, 2014-02-14 00:56:55]

On 2014/02/14 00:54:31, eseidel wrote:
> FYI, Frame ownership is moving into the embedder:
> https://codereview.chromium.org/156123004/

Don't know why I didn't think of that, I guess this whole approach may need some
rethinking, will CC you on the bug.

[adamk, 2014-02-14 01:39:56]

+dcheng for discussion of frame tree walking

[michaeln, 2014-02-14 02:12:46]

lgtm

> (though the ownership semantics aren't completely clear to me)

DocumentLoader owns (ApplicationCacheHost owns Internal owns
WebApplicationCacheHost)

And smushing the Internal class into its owner sgtm.

[eseidel, 2014-02-14 10:20:38]

The CQ bit was checked by eseidel@chromium.org

[commit-bot: I haz the power, 2014-02-14 10:20:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/adamk@chromium.org/161213003/30001

[commit-bot: I haz the power, 2014-02-14 10:39:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-02-14 10:39:28]

Retried try job too often on blink_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_pres...

[adamk, 2014-02-18 18:15:59]

ping abarth for public/ OWNERS

[abarth-chromium, 2014-02-18 18:20:17]

public/ LGTM

[adamk, 2014-02-18 18:25:21]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2014-02-18 18:25:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/adamk@chromium.org/161213003/30001

[commit-bot: I haz the power, 2014-02-18 20:31:21]

Change committed as 167359