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

Unified Diff: third_party/WebKit/Source/core/fetch/ResourceFetcher.h

Issue 2533683002: Move the code in ResourceFetcher handling calls from WebURLLoaderImpl to ResourceLoader (Closed)
Patch Set: Fix test Created 4 years 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
Index: third_party/WebKit/Source/core/fetch/ResourceFetcher.h
diff --git a/third_party/WebKit/Source/core/fetch/ResourceFetcher.h b/third_party/WebKit/Source/core/fetch/ResourceFetcher.h
index 089727626338a02134d9e32c96c881d2e793be41..c4b2df5b5cea52b952c563d25e29df7fee75fa56 100644
--- a/third_party/WebKit/Source/core/fetch/ResourceFetcher.h
+++ b/third_party/WebKit/Source/core/fetch/ResourceFetcher.h
@@ -115,24 +115,13 @@ class CORE_EXPORT ResourceFetcher
void stopFetching();
bool isFetching() const;
- ResourceRequestBlockedReason willFollowRedirect(Resource*,
- ResourceRequest&,
- const ResourceResponse&);
- enum DidFinishLoadingReason {
- DidFinishLoading,
- DidFinishFirstPartInMultipart
- };
- void didFinishLoading(Resource*,
- double finishTime,
- DidFinishLoadingReason);
- void didFailLoading(Resource*, const ResourceError&);
- void didReceiveResponse(Resource*,
- const ResourceResponse&,
- std::unique_ptr<WebDataConsumerHandle>);
- void didReceiveData(const Resource*, const char* data, int dataLength);
- void didReceiveTransferSizeUpdate(const Resource*, int transferSizeDiff);
- void didDownloadData(const Resource*, int dataLength, int encodedDataLength);
- bool defersLoading() const;
+ bool shouldDeferImageLoad(const KURL&) const;
+
+ void recordResourceTimingOnRedirect(Resource*, const ResourceResponse&, bool);
+
+ enum LoaderFinishType { DidFinishLoading, DidFinishFirstPartInMultipart };
+ void handleLoaderFinish(Resource*, double finishTime, LoaderFinishType);
+ void handleLoaderError(Resource*, const ResourceError&);
bool isControlledByServiceWorker() const;
enum ResourceLoadStartType {
@@ -196,17 +185,11 @@ class CORE_EXPORT ResourceFetcher
void moveCachedNonBlockingResourceToBlocking(Resource*, const FetchRequest&);
void moveResourceLoaderToNonBlocking(ResourceLoader*);
void removeResourceLoader(ResourceLoader*);
+ void handleLoadCompletion(Resource*);
void initializeResourceRequest(ResourceRequest&,
Resource::Type,
FetchRequest::DeferOption);
- void willSendRequest(unsigned long identifier,
- ResourceRequest&,
- const ResourceResponse&,
- const ResourceLoaderOptions&);
- ResourceRequestBlockedReason canAccessResponse(Resource*,
- const ResourceResponse&) const;
-
void requestLoadStarted(unsigned long identifier,
Resource*,
const FetchRequest&,
@@ -214,7 +197,6 @@ class CORE_EXPORT ResourceFetcher
bool isStaticData = false);
bool resourceNeedsLoad(Resource*, const FetchRequest&, RevalidationPolicy);
- bool shouldDeferImageLoad(const KURL&) const;
void resourceTimingReportTimerFired(TimerBase*);
« no previous file with comments | « third_party/WebKit/Source/core/fetch/FetchContext.h ('k') | third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698