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

Unified Diff: trunk/Source/core/loader/cache/CachedResourceLoader.cpp

Issue 19683007: Revert 154452 "Refactoring: Introduce ResouceLoaderHost interfac..." (Closed) Base URL: svn://svn.chromium.org/blink/
Patch Set: Created 7 years, 5 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 | « trunk/Source/core/loader/cache/CachedResourceLoader.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/Source/core/loader/cache/CachedResourceLoader.cpp
===================================================================
--- trunk/Source/core/loader/cache/CachedResourceLoader.cpp (revision 154456)
+++ trunk/Source/core/loader/cache/CachedResourceLoader.cpp (working copy)
@@ -31,13 +31,11 @@
#include "core/dom/Document.h"
#include "core/html/HTMLElement.h"
#include "core/html/HTMLFrameOwnerElement.h"
-#include "core/inspector/InspectorInstrumentation.h"
#include "core/loader/DocumentLoader.h"
#include "core/loader/FrameLoader.h"
#include "core/loader/FrameLoaderClient.h"
#include "core/loader/PingLoader.h"
#include "core/loader/UniqueIdentifier.h"
-#include "core/loader/appcache/ApplicationCacheHost.h"
#include "core/loader/cache/CachedCSSStyleSheet.h"
#include "core/loader/cache/CachedDocument.h"
#include "core/loader/cache/CachedFont.h"
@@ -502,7 +500,7 @@
if (!frame())
return 0;
- FrameLoader* frameLoader = this->frameLoader();
+ FrameLoader* frameLoader = frame()->loader();
if (request.options().securityCheck == DoSecurityCheck && (frameLoader->state() == FrameStateProvisional || !frameLoader->activeDocumentLoader() || frameLoader->activeDocumentLoader()->isStopping()))
return 0;
@@ -895,7 +893,7 @@
return CachePolicyVerify;
}
-void CachedResourceLoader::didLoadResource(CachedResource* resource)
+void CachedResourceLoader::loadDone(CachedResource* resource)
{
RefPtr<DocumentLoader> protectDocumentLoader(m_documentLoader);
RefPtr<Document> protectDocument(m_document);
@@ -1079,120 +1077,6 @@
m_pendingPreloads.clear();
}
-inline FrameLoader* CachedResourceLoader::frameLoader()
-{
- return frame() ? frame()->loader() : 0;
-}
-
-void CachedResourceLoader::didFinishLoading(const CachedResource* resource, double finishTime, const ResourceLoaderOptions& options)
-{
- if (options.sendLoadCallbacks != SendCallbacks)
- return;
- if (FrameLoader* loader = frameLoader())
- loader->notifier()->dispatchDidFinishLoading(m_documentLoader, resource->identifier(), finishTime);
-}
-
-void CachedResourceLoader::didChangeLoadingPriority(const CachedResource* resource, ResourceLoadPriority loadPriority)
-{
- if (FrameLoader* loader = frameLoader())
- loader->client()->dispatchDidChangeResourcePriority(resource->identifier(), loadPriority);
-}
-
-void CachedResourceLoader::didFailLoading(const CachedResource* resource, const ResourceError& error, const ResourceLoaderOptions& options)
-{
- if (options.sendLoadCallbacks != SendCallbacks)
- return;
- if (FrameLoader* loader = frameLoader())
- loader->notifier()->dispatchDidFail(m_documentLoader, resource->identifier(), error);
-}
-
-void CachedResourceLoader::willSendRequest(const CachedResource* resource, ResourceRequest& request, const ResourceResponse& redirectResponse, const ResourceLoaderOptions& options)
-{
- if (options.sendLoadCallbacks == SendCallbacks) {
- if (FrameLoader* loader = frameLoader())
- loader->notifier()->dispatchWillSendRequest(m_documentLoader, resource->identifier(), request, redirectResponse, options.initiatorInfo);
- } else {
- InspectorInstrumentation::willSendRequest(frame(), resource->identifier(), m_documentLoader, request, redirectResponse, options.initiatorInfo);
- }
-}
-
-void CachedResourceLoader::didReceiveResponse(const CachedResource* resource, const ResourceResponse& response, const ResourceLoaderOptions& options)
-{
- if (options.sendLoadCallbacks != SendCallbacks)
- return;
- if (FrameLoader* loader = frameLoader())
- loader->notifier()->dispatchDidReceiveResponse(m_documentLoader, resource->identifier(), response);
-}
-
-void CachedResourceLoader::didReceiveData(const CachedResource* resource, const char* data, int dataLength, int encodedDataLength, const ResourceLoaderOptions& options)
-{
- if (options.sendLoadCallbacks != SendCallbacks)
- return;
- if (FrameLoader* loader = frameLoader())
- loader->notifier()->dispatchDidReceiveData(m_documentLoader, resource->identifier(), data, dataLength, encodedDataLength);
-}
-
-void CachedResourceLoader::subresourceLoaderFinishedLoadingOnePart(ResourceLoader* loader)
-{
- if (m_documentLoader)
- m_documentLoader->subresourceLoaderFinishedLoadingOnePart(loader);
-}
-
-void CachedResourceLoader::didInitializeResourceLoader(ResourceLoader* loader)
-{
- if (m_documentLoader)
- m_documentLoader->addResourceLoader(loader);
-}
-
-void CachedResourceLoader::willTerminateResourceLoader(ResourceLoader* loader)
-{
- if (m_documentLoader)
- m_documentLoader->removeResourceLoader(loader);
-}
-
-void CachedResourceLoader::willStartLoadingResource(ResourceRequest& request)
-{
- if (m_documentLoader)
- m_documentLoader->applicationCacheHost()->willStartLoadingResource(request);
-}
-
-bool CachedResourceLoader::defersLoading() const
-{
- if (Frame* frame = this->frame())
- return frame->page()->defersLoading();
- return false;
-}
-
-bool CachedResourceLoader::isLoadedBy(ResourceLoaderHost* possibleOwner) const
-{
- return this == possibleOwner;
-}
-
-bool CachedResourceLoader::shouldRequest(CachedResource* resource, const ResourceRequest& request, const ResourceLoaderOptions& options)
-{
- if (!canRequest(resource->type(), request.url(), options))
- return false;
- if (resource->type() == CachedResource::ImageResource && shouldDeferImageLoad(request.url()))
- return false;
- return true;
-}
-
-Frame* CachedResourceLoader::inspectedFrame() const
-{
- // FIXME: return frame of master document for imported documents.
- return m_documentLoader ? m_documentLoader->frame() : 0;
-}
-
-void CachedResourceLoader::refResourceLoaderHost()
-{
- ref();
-}
-
-void CachedResourceLoader::derefResourceLoaderHost()
-{
- deref();
-}
-
#if PRELOAD_DEBUG
void CachedResourceLoader::printPreloadStats()
{
« no previous file with comments | « trunk/Source/core/loader/cache/CachedResourceLoader.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698