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

Unified Diff: third_party/WebKit/Source/core/loader/FrameFetchContext.cpp

Issue 2768293002: FrameFetchContext should respect BypassingCache (Closed)
Patch Set: rebase Created 3 years, 9 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 | third_party/WebKit/Source/core/loader/FrameFetchContextTest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/loader/FrameFetchContext.cpp
diff --git a/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp b/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp
index 2c93dbb8c59575e1ac5ea652bd87b831bfdf4284..5c636ca93c8302350ff3ae426650a4db62c83afc 100644
--- a/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp
+++ b/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp
@@ -257,13 +257,7 @@ WebCachePolicy determineWebCachePolicy(RequestMethod method,
? WebCachePolicy::ValidatingCacheData
: WebCachePolicy::UseProtocolCachePolicy;
case FrameLoadTypeReloadBypassingCache:
- // TODO(toyoshim): Should return BypassingCache always, but keep legacy
- // logic as is. To be changed in a follow-up patch soon.
- return (resourceType == ResourceType::kIsMainResource &&
- (requestType == RequestType::kIsConditional ||
- method == RequestMethod::kIsPost))
- ? WebCachePolicy::ValidatingCacheData
- : WebCachePolicy::BypassingCache;
+ return WebCachePolicy::BypassingCache;
}
NOTREACHED();
return WebCachePolicy::UseProtocolCachePolicy;
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/loader/FrameFetchContextTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698