Introduce url_list to the Response scheme of CacheStorage.

content/common/service_worker/service_worker_types.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/common/service_worker/service_worker_types.h"
 
 namespace content {
 
 const char kServiceWorkerRegisterErrorPrefix[] =
     "Failed to register a ServiceWorker: ";
@@ skipping 58 matching lines @@
   return size;
 }
 
 ServiceWorkerResponse::ServiceWorkerResponse()
     : status_code(0),
       response_type(blink::WebServiceWorkerResponseTypeOpaque),
       blob_size(0),
       error(blink::WebServiceWorkerResponseErrorUnknown) {}
 
 ServiceWorkerResponse::ServiceWorkerResponse(
-    const GURL& url,
+    std::unique_ptr<std::vector<GURL>> url_list,

[falken, 2016/11/30 14:57:37]

just to be sure: the reason for changing the const& to unique_ptr here and below
is to reduce copying?

[horo, 2016/12/01 07:42:13]

yes.

     int status_code,
     const std::string& status_text,
     blink::WebServiceWorkerResponseType response_type,
-    const ServiceWorkerHeaderMap& headers,
+    std::unique_ptr<ServiceWorkerHeaderMap> headers,
     const std::string& blob_uuid,
     uint64_t blob_size,
     const GURL& stream_url,
     blink::WebServiceWorkerResponseError error,
     base::Time response_time,
     bool is_in_cache_storage,
     const std::string& cache_storage_cache_name,
-    const ServiceWorkerHeaderList& cors_exposed_headers)
-    : url(url),
-      status_code(status_code),
+    std::unique_ptr<ServiceWorkerHeaderList> cors_exposed_headers)
+    : status_code(status_code),
       status_text(status_text),
       response_type(response_type),
-      headers(headers),
       blob_uuid(blob_uuid),
       blob_size(blob_size),
       stream_url(stream_url),
       error(error),
       response_time(response_time),
       is_in_cache_storage(is_in_cache_storage),
-      cache_storage_cache_name(cache_storage_cache_name),
-      cors_exposed_header_names(cors_exposed_headers) {}
+      cache_storage_cache_name(cache_storage_cache_name) {
+  this->url_list.swap(*url_list);
+  this->headers.swap(*headers);
+  this->cors_exposed_header_names.swap(*cors_exposed_headers);
+}
 
 ServiceWorkerResponse::ServiceWorkerResponse(
     const ServiceWorkerResponse& other) = default;
 
 ServiceWorkerResponse::~ServiceWorkerResponse() {}
 
 size_t ServiceWorkerResponse::EstimatedStructSize() {
   size_t size = sizeof(ServiceWorkerResponse);
-  size += url.spec().size();
+  for (const auto& url : url_list)
+    size += url.spec().size();
   size += blob_uuid.size();
   size += stream_url.spec().size();
   size += cache_storage_cache_name.size();
   for (const auto& key_and_value : headers) {
     size += key_and_value.first.size();
     size += key_and_value.second.size();
   }
   for (const auto& header : cors_exposed_header_names)
     size += header.size();
   return size;
@@ skipping 32 matching lines @@
 NavigationPreloadState::NavigationPreloadState()
     : enabled(false), header("true") {}
 
 NavigationPreloadState::NavigationPreloadState(bool enabled, std::string header)
     : enabled(enabled), header(header) {}
 
 NavigationPreloadState::NavigationPreloadState(
     const NavigationPreloadState& other) = default;
 
 }  // namespace content