Introduce url_list to the Response scheme of CacheStorage.

third_party/WebKit/Source/modules/fetch/Response.cpp

 // 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 "modules/fetch/Response.h"
 
 #include "bindings/core/v8/Dictionary.h"
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "bindings/core/v8/V8ArrayBuffer.h"
@@ skipping 26 matching lines @@
 
 FetchResponseData* createFetchResponseDataFromWebResponse(
     ScriptState* scriptState,
     const WebServiceWorkerResponse& webResponse) {
   FetchResponseData* response = nullptr;
   if (webResponse.status() > 0)
     response = FetchResponseData::create();
   else
     response = FetchResponseData::createNetworkErrorResponse();
 
-  response->setURL(webResponse.url());
+  const blink::WebVector<blink::WebURL>& webURLList = webResponse.urlList();
+  Vector<blink::KURL> urlList(webURLList.size());
+  std::transform(webURLList.begin(), webURLList.end(), urlList.begin(),
+                 [](const blink::WebURL& url) { return url; });

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

we shouldn't need all the blink:: here, no?

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

Done.

+  response->setURLList(urlList);
   response->setStatus(webResponse.status());
   response->setStatusMessage(webResponse.statusText());
   response->setResponseTime(webResponse.responseTime());
   response->setCacheStorageCacheName(webResponse.cacheStorageCacheName());
 
   for (HTTPHeaderMap::const_iterator i = webResponse.headers().begin(),
                                      end = webResponse.headers().end();
        i != end; ++i) {
     response->headerList()->append(i->key, i->value);
   }
@@ skipping 285 matching lines @@
       return "opaqueredirect";
   }
   ASSERT_NOT_REACHED();
   return "";
 }
 
 String Response::url() const {
   // "The url attribute's getter must return the empty string if response's
   // url is null and response's url, serialized with the exclude fragment
   // flag set, otherwise."
-  if (!m_response->url().hasFragmentIdentifier())
-    return m_response->url();
-  KURL url(m_response->url());
+  const KURL* responseURL = m_response->url();
+  if (!responseURL)
+    return emptyString();
+  if (!responseURL->hasFragmentIdentifier())
+    return *responseURL;
+  KURL url(*responseURL);
   url.removeFragmentIdentifier();
   return url;
 }
 
 unsigned short Response::status() const {
   // "The status attribute's getter must return response's status."
   return m_response->status();
 }
 
 bool Response::ok() const {
@@ skipping 95 matching lines @@
       V8HiddenValue::internalBodyBuffer(scriptState->isolate()), bodyBuffer);
 }
 
 DEFINE_TRACE(Response) {
   Body::trace(visitor);
   visitor->trace(m_response);
   visitor->trace(m_headers);
 }
 
 }  // namespace blink