Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1409)

Unified Diff: Source/core/loader/appcache/ApplicationCacheHostInternal.h

Issue 161213003: Fix layering of ApplicationCacheHost (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: The real thing Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/loader/appcache/ApplicationCacheHost.cpp ('k') | Source/web/ApplicationCacheHost.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/loader/appcache/ApplicationCacheHostInternal.h
diff --git a/Source/web/ApplicationCacheHostInternal.h b/Source/core/loader/appcache/ApplicationCacheHostInternal.h
similarity index 85%
rename from Source/web/ApplicationCacheHostInternal.h
rename to Source/core/loader/appcache/ApplicationCacheHostInternal.h
index c77f922fff73a898dfe018c39fd2e69970bd84f2..c614d866bcb16eaddef08c39cadeadb17e8f5538 100644
--- a/Source/web/ApplicationCacheHostInternal.h
+++ b/Source/core/loader/appcache/ApplicationCacheHostInternal.h
@@ -28,23 +28,26 @@
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
-#include "WebApplicationCacheHostClient.h"
-#include "WebFrameClient.h"
-#include "WebFrameImpl.h"
+#include "core/frame/Frame.h"
#include "core/loader/DocumentLoader.h"
+#include "core/loader/FrameLoader.h"
+#include "core/loader/FrameLoaderClient.h"
#include "core/loader/appcache/ApplicationCacheHost.h"
+#include "public/platform/WebApplicationCacheHostClient.h"
#include "public/platform/WebURL.h"
namespace WebCore {
+// FIXME: This class should go away and WebCore::ApplicationCacheHost should implement blink::WebApplicationCacheHostClient
class ApplicationCacheHostInternal FINAL : public blink::WebApplicationCacheHostClient {
public:
ApplicationCacheHostInternal(ApplicationCacheHost* host)
: m_innerHost(host)
{
- blink::WebFrameImpl* webFrame = blink::WebFrameImpl::fromFrame(host->m_documentLoader->frame());
- ASSERT(webFrame);
- m_outerHost = adoptPtr(webFrame->client()->createApplicationCacheHost(webFrame, this));
+ Frame* frame = host->m_documentLoader->frame();
+ ASSERT(frame);
+ ASSERT(frame->loader().client());
+ m_outerHost = frame->loader().client()->createApplicationCacheHost(this);
}
virtual void didChangeCacheAssociation() OVERRIDE
« no previous file with comments | « Source/core/loader/appcache/ApplicationCacheHost.cpp ('k') | Source/web/ApplicationCacheHost.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698