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

Unified Diff: content/browser/cache_storage/cache_storage_cache.cc

Issue 2785123002: Make no-location redirect response to be "opaque redirect" when redirect mode is manual. (Closed)
Patch Set: add link to crbug.com/707185 Created 3 years, 8 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/LayoutTests/external/wpt/fetch/api/redirect/redirect-location.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/cache_storage/cache_storage_cache.cc
diff --git a/content/browser/cache_storage/cache_storage_cache.cc b/content/browser/cache_storage/cache_storage_cache.cc
index 398792fac8387ff83f173e2f5cdb9bbe4c212053..4bb7598563e052374ff273ecd77c91e66c4ee5f7 100644
--- a/content/browser/cache_storage/cache_storage_cache.cc
+++ b/content/browser/cache_storage/cache_storage_cache.cc
@@ -1019,10 +1019,6 @@ void CacheStorageCache::Put(const CacheStorageBatchOperation& operation,
// We don't support streaming for cache.
DCHECK(operation.response.stream_url.is_empty());
- // We don't support the body of redirect response.
- DCHECK(!(operation.response.response_type ==
- blink::kWebServiceWorkerResponseTypeOpaqueRedirect &&
- operation.response.blob_size));
std::unique_ptr<ServiceWorkerResponse> response =
base::MakeUnique<ServiceWorkerResponse>(operation.response);
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/external/wpt/fetch/api/redirect/redirect-location.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698