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

Unified Diff: content/renderer/cache_storage/cache_storage_dispatcher.cc

Issue 2516353002: Introduce url_list to the Response scheme of CacheStorage. (Closed)
Patch Set: implicit conversion WebURL <-> GURL and WebVector <- vector 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: content/renderer/cache_storage/cache_storage_dispatcher.cc
diff --git a/content/renderer/cache_storage/cache_storage_dispatcher.cc b/content/renderer/cache_storage/cache_storage_dispatcher.cc
index 99c1605180a409fd0497b893dab1339399a69748..01814f2d1bd8fffd04cdd67244fa62c3ec93d3aa 100644
--- a/content/renderer/cache_storage/cache_storage_dispatcher.cc
+++ b/content/renderer/cache_storage/cache_storage_dispatcher.cc
@@ -79,25 +79,6 @@ blink::WebVector<blink::WebServiceWorkerRequest> WebRequestsFromRequests(
return web_requests;
}
-ServiceWorkerResponse ResponseFromWebResponse(
- const blink::WebServiceWorkerResponse& web_response) {
- ServiceWorkerHeaderMap headers;
- GetServiceWorkerHeaderMapFromWebResponse(web_response, &headers);
- ServiceWorkerHeaderList cors_exposed_header_names;
- GetCorsExposedHeaderNamesFromWebResponse(web_response,
- &cors_exposed_header_names);
- // We don't support streaming for cache.
- DCHECK(web_response.streamURL().isEmpty());
- return ServiceWorkerResponse(
- web_response.url(), web_response.status(),
- web_response.statusText().ascii(), web_response.responseType(), headers,
- web_response.blobUUID().ascii(), web_response.blobSize(),
- web_response.streamURL(), blink::WebServiceWorkerResponseErrorUnknown,
- base::Time::FromInternalValue(web_response.responseTime()),
- !web_response.cacheStorageCacheName().isNull(),
- web_response.cacheStorageCacheName().utf8(), cors_exposed_header_names);
-}
-
CacheStorageCacheQueryParams QueryParamsFromWebQueryParams(
const blink::WebServiceWorkerCache::QueryParams& web_query_params) {
CacheStorageCacheQueryParams query_params;
@@ -122,11 +103,15 @@ CacheStorageCacheOperationType CacheOperationTypeFromWebCacheOperationType(
CacheStorageBatchOperation BatchOperationFromWebBatchOperation(
const blink::WebServiceWorkerCache::BatchOperation& web_operation) {
+ // We don't support streaming for cache.
+ DCHECK(web_operation.response.streamURL().isEmpty());
+
CacheStorageBatchOperation operation;
operation.operation_type =
CacheOperationTypeFromWebCacheOperationType(web_operation.operationType);
operation.request = FetchRequestFromWebRequest(web_operation.request);
- operation.response = ResponseFromWebResponse(web_operation.response);
+ operation.response =
+ GetServiceWorkerResponseFromWebResponse(web_operation.response);
operation.match_params =
QueryParamsFromWebQueryParams(web_operation.matchParams);
return operation;
@@ -643,7 +628,7 @@ void CacheStorageDispatcher::OnWebCacheDestruction(int cache_id) {
void CacheStorageDispatcher::PopulateWebResponseFromResponse(
const ServiceWorkerResponse& response,
blink::WebServiceWorkerResponse* web_response) {
- web_response->setURL(response.url);
+ web_response->setURLList(response.url_list);
web_response->setStatus(response.status_code);
web_response->setStatusText(WebString::fromASCII(response.status_text));
web_response->setResponseType(response.response_type);
« no previous file with comments | « content/public/common/resource_response_info.h ('k') | content/renderer/service_worker/service_worker_context_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698