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

Unified Diff: content/browser/appcache/appcache_request_handler.h

Issue 2982363002: Add support for fallback content for the frame. This includes main and subframes. (Closed)
Patch Set: rebase to tip Created 3 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 | « no previous file | content/browser/appcache/appcache_request_handler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/appcache/appcache_request_handler.h
diff --git a/content/browser/appcache/appcache_request_handler.h b/content/browser/appcache/appcache_request_handler.h
index 24ee0fcc7456cbd50af6d75a4c43e522325a6630..439e461371e31901a00720592f41d6776a1f04a2 100644
--- a/content/browser/appcache/appcache_request_handler.h
+++ b/content/browser/appcache/appcache_request_handler.h
@@ -160,11 +160,19 @@ class CONTENT_EXPORT AppCacheRequestHandler
// AppCacheHost::Observer override
void OnCacheSelectionComplete(AppCacheHost* host) override;
- // URLLoaderRequestHandler override
+ // Network service loading
+
+ // URLLoaderRequestHandler overrides
+ // These functions are invoked for loading AppCache content for the frame and
+ // for subresources.
void MaybeCreateLoader(const ResourceRequest& resource_request,
ResourceContext* resource_context,
LoaderCallback callback) override;
mojom::URLLoaderFactoryPtr MaybeCreateSubresourceFactory() override;
+ bool MaybeCreateLoaderForResponse(
+ const ResourceResponseHead& response,
+ mojom::URLLoaderPtr* loader,
+ mojom::URLLoaderClientRequest* client_request) override;
// Data members -----------------------------------------------
@@ -235,8 +243,7 @@ class CONTENT_EXPORT AppCacheRequestHandler
// the AppCache, we delete the job.
std::unique_ptr<AppCacheJob> navigation_request_job_;
- // In the network service world, points to the getter for the network URL
- // loader.
+ // Points to the getter for the network URL loader.
scoped_refptr<URLLoaderFactoryGetter> network_url_loader_factory_getter_;
friend class content::AppCacheRequestHandlerTest;
« no previous file with comments | « no previous file | content/browser/appcache/appcache_request_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698