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

Unified Diff: content/browser/appcache/appcache_interceptor.cc

Issue 2501343003: PlzNavigate: AppCache support. (Closed)
Patch Set: Add DCHECKs for PlzNavigate and fix a double Release problem which caused one unit_test to fail wit… Created 4 years, 1 month 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: content/browser/appcache/appcache_interceptor.cc
diff --git a/content/browser/appcache/appcache_interceptor.cc b/content/browser/appcache/appcache_interceptor.cc
index 85022442562c6f188f524bd50fabdf98e3b33f3b..8d2d93651c99e5e710f0d598cfbe74426a84062f 100644
--- a/content/browser/appcache/appcache_interceptor.cc
+++ b/content/browser/appcache/appcache_interceptor.cc
@@ -31,19 +31,22 @@ AppCacheRequestHandler* AppCacheInterceptor::GetHandler(
request->GetUserData(&kHandlerKey));
}
-void AppCacheInterceptor::SetExtraRequestInfo(
- net::URLRequest* request,
- AppCacheServiceImpl* service,
- int process_id,
- int host_id,
- ResourceType resource_type,
- bool should_reset_appcache) {
+void AppCacheInterceptor::SetExtraRequestInfo(net::URLRequest* request,
+ AppCacheServiceImpl* service,
+ int process_id,
+ int frame_id,
+ int host_id,
+ ResourceType resource_type,
+ bool should_reset_appcache) {
if (!service || (host_id == kAppCacheNoHostId))
return;
AppCacheBackendImpl* backend = service->GetBackend(process_id);
- if (!backend)
- return;
+ if (!backend) {
+ backend = service->GetBackendForFrame(frame_id);
+ if (!backend)
+ return;
+ }
// TODO(michaeln): An invalid host id is indicative of bad data
// from a child process. How should we handle that here?

Powered by Google App Engine
This is Rietveld 408576698