service worker: Remove dead code around ServiceWorkerResponseInfo.

This CL removes some dead code following the launch of
S13nServiceWorker:
* OnPrepareToRestart() for navigations which has no callsites.
* ServiceWorkerResponseInfo which was only possibly created in
  ForRequest(), but all callsites now pass |create| = false.

This removes code from ClearSiteDataThrottle which has not been
needed since S13nServiceWorker. The requests directly handled
and responded to by the SW no longer reach
ClearSiteDataThrottle::HandleHeader() at all. Furthermore, when
the network service is enabled, this file will be deleted as
ClearSideDataHandler handles the Clear-Site-Data directive.
The test for this is ClearSiteDataHandlerBrowserTest.ServiceWorker
which still passes before and after this CL.

Bug: 926114
Change-Id: Ib2a221a3711bdaaa89b7ddd3771ac8bf3bbca314
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1567149
Reviewed-by: Christian Dullweber <dullweber@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#651196}
diff --git a/content/browser/BUILD.gn b/content/browser/BUILD.gn
index 762a38f..115d7c5 100644
--- a/content/browser/BUILD.gn
+++ b/content/browser/BUILD.gn
@@ -1736,8 +1736,6 @@
     "service_worker/service_worker_registration_status.h",
     "service_worker/service_worker_request_handler.cc",
     "service_worker/service_worker_request_handler.h",
-    "service_worker/service_worker_response_info.cc",
-    "service_worker/service_worker_response_info.h",
     "service_worker/service_worker_script_cache_map.cc",
     "service_worker/service_worker_script_cache_map.h",
     "service_worker/service_worker_script_loader_factory.cc",
diff --git a/content/browser/browsing_data/clear_site_data_throttle.cc b/content/browser/browsing_data/clear_site_data_throttle.cc
index 00d130d..87784c1 100644
--- a/content/browser/browsing_data/clear_site_data_throttle.cc
+++ b/content/browser/browsing_data/clear_site_data_throttle.cc
@@ -15,7 +15,6 @@
 #include "base/task/post_task.h"
 #include "base/values.h"
 #include "content/browser/renderer_host/render_process_host_impl.h"
-#include "content/browser/service_worker/service_worker_response_info.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_task_traits.h"
@@ -278,27 +277,6 @@
     return false;
   }
 
-  // Service workers can handle fetches of third-party resources and inject
-  // arbitrary headers. Ignore responses that came from a service worker,
-  // as supporting Clear-Site-Data would give them the power to delete data from
-  // any website.
-  // See https://w3c.github.io/webappsec-clear-site-data/#service-workers
-  // for more information.
-  const ServiceWorkerResponseInfo* response_info =
-      ServiceWorkerResponseInfo::ForRequest(request_);
-  if (response_info) {
-    network::ResourceResponseInfo extra_response_info;
-    response_info->GetExtraResponseInfo(&extra_response_info);
-
-    if (extra_response_info.was_fetched_via_service_worker) {
-      delegate_->AddMessage(
-          GetCurrentURL(),
-          "Ignoring, as the response came from a service worker.",
-          blink::mojom::ConsoleMessageLevel::kError);
-      return false;
-    }
-  }
-
   bool clear_cookies;
   bool clear_storage;
   bool clear_cache;
diff --git a/content/browser/loader/resource_loader.cc b/content/browser/loader/resource_loader.cc
index bd0f57f..ac3e5c8 100644
--- a/content/browser/loader/resource_loader.cc
+++ b/content/browser/loader/resource_loader.cc
@@ -22,7 +22,6 @@
 #include "content/browser/loader/resource_loader_delegate.h"
 #include "content/browser/loader/resource_request_info_impl.h"
 #include "content/browser/service_worker/service_worker_request_handler.h"
-#include "content/browser/service_worker/service_worker_response_info.h"
 #include "content/browser/ssl/ssl_client_auth_handler.h"
 #include "content/browser/ssl/ssl_manager.h"
 #include "content/public/browser/content_browser_client.h"
@@ -110,10 +109,6 @@
     }
   }
 
-  const ServiceWorkerResponseInfo* service_worker_info =
-      ServiceWorkerResponseInfo::ForRequest(request);
-  if (service_worker_info)
-    service_worker_info->GetExtraResponseInfo(&response->head);
   response->head.appcache_id = blink::mojom::kAppCacheNoCacheId;
   AppCacheInterceptor::GetExtraResponseInfo(
       request, &response->head.appcache_id,
diff --git a/content/browser/service_worker/service_worker_controllee_request_handler.cc b/content/browser/service_worker/service_worker_controllee_request_handler.cc
index b67829d..45426aa 100644
--- a/content/browser/service_worker/service_worker_controllee_request_handler.cc
+++ b/content/browser/service_worker/service_worker_controllee_request_handler.cc
@@ -15,7 +15,6 @@
 #include "content/browser/service_worker/service_worker_navigation_loader.h"
 #include "content/browser/service_worker/service_worker_provider_host.h"
 #include "content/browser/service_worker/service_worker_registration.h"
-#include "content/browser/service_worker/service_worker_response_info.h"
 #include "content/common/navigation_subresource_loader_params.h"
 #include "content/common/service_worker/service_worker_utils.h"
 #include "content/public/browser/content_browser_client.h"
@@ -116,7 +115,6 @@
       frame_type_(frame_type),
       body_(std::move(body)),
       force_update_started_(false),
-      use_network_(false),
       weak_factory_(this) {
   DCHECK(ServiceWorkerUtils::IsMainResourceType(resource_type));
 }
@@ -160,10 +158,6 @@
     return;
   }
 
-  // In fallback cases we basically 'forward' the request, so we should
-  // never see use_network_ gets true.
-  DCHECK(!use_network_);
-
 #if BUILDFLAG(ENABLE_OFFLINE_PAGES)
   // Fall back for the subsequent offline page interceptor to load the offline
   // snapshot of the page if required.
@@ -542,11 +536,6 @@
       std::move(disallow_controller)));
 }
 
-void ServiceWorkerControlleeRequestHandler::OnPrepareToRestart() {
-  use_network_ = true;
-  ClearJob();
-}
-
 ServiceWorkerVersion*
 ServiceWorkerControlleeRequestHandler::GetServiceWorkerVersion(
     ServiceWorkerMetrics::URLRequestJobResult* result) {
diff --git a/content/browser/service_worker/service_worker_controllee_request_handler.h b/content/browser/service_worker/service_worker_controllee_request_handler.h
index 5ff9416..ab02c38 100644
--- a/content/browser/service_worker/service_worker_controllee_request_handler.h
+++ b/content/browser/service_worker/service_worker_controllee_request_handler.h
@@ -107,7 +107,6 @@
           disallow_controller);
 
   // ServiceWorkerNavigationLoader::Delegate implementation:
-  void OnPrepareToRestart() override;
   ServiceWorkerVersion* GetServiceWorkerVersion(
       ServiceWorkerMetrics::URLRequestJobResult* result) override;
   bool RequestStillValid(
@@ -136,11 +135,6 @@
   GURL stripped_url_;
   bool force_update_started_;
 
-  // True if the next time this request is started, the response should be
-  // delivered from the network, bypassing the ServiceWorker. Cleared after the
-  // next intercept opportunity, for main frame requests.
-  bool use_network_;
-
   base::WeakPtrFactory<ServiceWorkerControlleeRequestHandler> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerControlleeRequestHandler);
diff --git a/content/browser/service_worker/service_worker_navigation_loader.h b/content/browser/service_worker/service_worker_navigation_loader.h
index 18b8255..ddbd19e 100644
--- a/content/browser/service_worker/service_worker_navigation_loader.h
+++ b/content/browser/service_worker/service_worker_navigation_loader.h
@@ -48,11 +48,6 @@
    public:
     virtual ~Delegate() {}
 
-    // Will be invoked before the request is restarted. The caller can use this
-    // opportunity to grab state from the ServiceWorkerNavigationLoader to
-    // determine how it should behave when the request is restarted.
-    virtual void OnPrepareToRestart() = 0;
-
     // Returns the ServiceWorkerVersion fetch events for this request job should
     // be dispatched to. If no appropriate worker can be determined, returns
     // nullptr and sets |*result| to an appropriate error.
diff --git a/content/browser/service_worker/service_worker_navigation_loader_unittest.cc b/content/browser/service_worker/service_worker_navigation_loader_unittest.cc
index a1c4a98..2334ebc 100644
--- a/content/browser/service_worker/service_worker_navigation_loader_unittest.cc
+++ b/content/browser/service_worker/service_worker_navigation_loader_unittest.cc
@@ -475,8 +475,6 @@
 
  protected:
   // ServiceWorkerNavigationLoader::Delegate -----------------------------------
-  void OnPrepareToRestart() override {}
-
   ServiceWorkerVersion* GetServiceWorkerVersion(
       ServiceWorkerMetrics::URLRequestJobResult* result) override {
     return version_.get();
diff --git a/content/browser/service_worker/service_worker_response_info.cc b/content/browser/service_worker/service_worker_response_info.cc
deleted file mode 100644
index e837a8d..0000000
--- a/content/browser/service_worker/service_worker_response_info.cc
+++ /dev/null
@@ -1,120 +0,0 @@
-// Copyright 2016 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/browser/service_worker/service_worker_response_info.h"
-
-#include "base/memory/ptr_util.h"
-#include "net/url_request/url_request.h"
-#include "services/network/public/cpp/resource_response_info.h"
-
-namespace content {
-
-// static
-int ServiceWorkerResponseInfo::user_data_key_;
-
-// static
-ServiceWorkerResponseInfo* ServiceWorkerResponseInfo::ForRequest(
-    net::URLRequest* request,
-    bool create) {
-  ServiceWorkerResponseInfo* info = static_cast<ServiceWorkerResponseInfo*>(
-      request->GetUserData(&user_data_key_));
-  if (!info && create) {
-    info = new ServiceWorkerResponseInfo();
-    request->SetUserData(&user_data_key_, base::WrapUnique(info));
-  }
-  return info;
-}
-
-// static
-void ServiceWorkerResponseInfo::ResetDataForRequest(net::URLRequest* request) {
-  ServiceWorkerResponseInfo* info = ForRequest(request);
-  if (info)
-    info->ResetData();
-}
-
-ServiceWorkerResponseInfo::~ServiceWorkerResponseInfo() {}
-
-void ServiceWorkerResponseInfo::GetExtraResponseInfo(
-    network::ResourceResponseInfo* response_info) const {
-  response_info->was_fetched_via_service_worker =
-      was_fetched_via_service_worker_;
-  response_info->was_fallback_required_by_service_worker =
-      was_fallback_required_;
-  response_info->url_list_via_service_worker = url_list_via_service_worker_;
-  response_info->response_type = response_type_via_service_worker_;
-  response_info->service_worker_start_time = service_worker_start_time_;
-  response_info->service_worker_ready_time = service_worker_ready_time_;
-  response_info->is_in_cache_storage = response_is_in_cache_storage_;
-  response_info->cache_storage_cache_name = response_cache_storage_cache_name_;
-  response_info->cors_exposed_header_names = cors_exposed_header_names_;
-  response_info->did_service_worker_navigation_preload =
-      did_navigation_preload_;
-}
-
-void ServiceWorkerResponseInfo::OnPrepareToRestart(
-    base::TimeTicks service_worker_start_time,
-    base::TimeTicks service_worker_ready_time,
-    bool did_navigation_preload) {
-  ResetData();
-
-  // Update times, if not already set by a previous Job.
-  if (service_worker_start_time_.is_null()) {
-    service_worker_start_time_ = service_worker_start_time;
-    service_worker_ready_time_ = service_worker_ready_time;
-  }
-  // Don't reset navigation preload flag it if a previous job already set it,
-  // since the UseCounter should still reflect that navigation preload occurred
-  // for this request.
-  if (did_navigation_preload)
-    did_navigation_preload_ = true;
-}
-
-void ServiceWorkerResponseInfo::OnStartCompleted(
-    bool was_fetched_via_service_worker,
-    bool was_fallback_required,
-    const std::vector<GURL>& url_list_via_service_worker,
-    network::mojom::FetchResponseType response_type_via_service_worker,
-    base::TimeTicks service_worker_start_time,
-    base::TimeTicks service_worker_ready_time,
-    bool response_is_in_cache_storage,
-    const std::string& response_cache_storage_cache_name,
-    const ServiceWorkerHeaderList& cors_exposed_header_names,
-    bool did_navigation_preload) {
-  was_fetched_via_service_worker_ = was_fetched_via_service_worker;
-  was_fallback_required_ = was_fallback_required;
-  url_list_via_service_worker_ = url_list_via_service_worker;
-  response_type_via_service_worker_ = response_type_via_service_worker;
-  response_is_in_cache_storage_ = response_is_in_cache_storage;
-  response_cache_storage_cache_name_ = response_cache_storage_cache_name;
-  cors_exposed_header_names_ = cors_exposed_header_names;
-
-  // Update times, if not already set by a previous Job.
-  if (service_worker_start_time_.is_null()) {
-    service_worker_start_time_ = service_worker_start_time;
-    service_worker_ready_time_ = service_worker_ready_time;
-  }
-
-  did_navigation_preload_ = did_navigation_preload;
-}
-
-void ServiceWorkerResponseInfo::ResetData() {
-  was_fetched_via_service_worker_ = false;
-  was_fallback_required_ = false;
-  url_list_via_service_worker_.clear();
-  response_type_via_service_worker_ =
-      network::mojom::FetchResponseType::kDefault;
-  // Don't reset |service_worker_start_time_| or |service_worker_ready_time_|
-  // since it's historical timing information that should persist between job
-  // restarts.
-  response_is_in_cache_storage_ = false;
-  response_cache_storage_cache_name_ = std::string();
-  cors_exposed_header_names_.clear();
-  // Don't reset the |did_navigation_preload_| flag. This is used for the
-  // UseCounter, and if it was ever true for a request, it should remain true
-  // even if the job restarts.
-}
-
-ServiceWorkerResponseInfo::ServiceWorkerResponseInfo() {}
-
-}  // namespace content
diff --git a/content/browser/service_worker/service_worker_response_info.h b/content/browser/service_worker/service_worker_response_info.h
deleted file mode 100644
index 187c8b9..0000000
--- a/content/browser/service_worker/service_worker_response_info.h
+++ /dev/null
@@ -1,118 +0,0 @@
-// Copyright 2016 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.
-
-#ifndef CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_RESPONSE_INFO_H_
-#define CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_RESPONSE_INFO_H_
-
-#include <vector>
-
-#include "base/supports_user_data.h"
-#include "content/common/content_export.h"
-#include "content/common/service_worker/service_worker_types.h"
-#include "services/network/public/mojom/fetch_api.mojom.h"
-#include "url/gurl.h"
-
-namespace net {
-class URLRequest;
-}
-
-namespace network {
-struct ResourceResponseInfo;
-}
-
-namespace content {
-
-
-class CONTENT_EXPORT ServiceWorkerResponseInfo
-    : public base::SupportsUserData::Data {
- public:
-  static ServiceWorkerResponseInfo* ForRequest(net::URLRequest* request,
-                                               bool create = false);
-  static void ResetDataForRequest(net::URLRequest* request);
-
-  ~ServiceWorkerResponseInfo() override;
-
-  void GetExtraResponseInfo(network::ResourceResponseInfo* response_info) const;
-  void OnPrepareToRestart(base::TimeTicks service_worker_start_time,
-                          base::TimeTicks service_worker_ready_time,
-                          bool did_navigation_preload);
-  void OnStartCompleted(
-      bool was_fetched_via_service_worker,
-      bool was_fallback_required,
-      const std::vector<GURL>& url_list_via_service_worker,
-      network::mojom::FetchResponseType response_type_via_service_worker,
-      base::TimeTicks service_worker_start_time,
-      base::TimeTicks service_worker_ready_time,
-      bool response_is_in_cache_storage,
-      const std::string& response_cache_storage_cache_name,
-      const ServiceWorkerHeaderList& cors_exposed_header_names,
-      bool did_navigation_preload);
-  void ResetData();
-
-  // Returns true if a service worker responded to the request. If the service
-  // worker received a fetch event and did not call respondWith(), or was
-  // bypassed due to absence of a fetch event handler, this function
-  // typically returns false but returns true if "fallback to renderer" was
-  // required (however in this case the response is not an actual resource and
-  // the request will be reissued by the renderer).
-  bool was_fetched_via_service_worker() const {
-    return was_fetched_via_service_worker_;
-  }
-
-  // Returns true if "fallback to renderer" was required. This happens when a
-  // request was directed to a service worker but it did not call respondWith()
-  // or was bypassed due to absense of a fetch event handler, and the browser
-  // could not directly fall back to network because the CORS algorithm must be
-  // run, which is implemented in the renderer.
-  bool was_fallback_required() const { return was_fallback_required_; }
-
-  // Returns the URL list of the Response object the service worker passed to
-  // respondWith() to create this response. For example, if the service worker
-  // calls respondWith(fetch('http://example.com/a')) and http://example.com/a
-  // redirects to http://example.net/b which redirects to http://example.org/c,
-  // the URL list is the vector <"http://example.com/a", "http://example.net/b",
-  // "http://example.org/c">. This is empty if the response was programatically
-  // generated as in respondWith(new Response()). It is also empty if a service
-  // worker did not respond to the request or did not call respondWith().
-  const std::vector<GURL>& url_list_via_service_worker() const {
-    return url_list_via_service_worker_;
-  }
-  network::mojom::FetchResponseType response_type_via_service_worker() const {
-    return response_type_via_service_worker_;
-  }
-  base::TimeTicks service_worker_start_time() const {
-    return service_worker_start_time_;
-  }
-  base::TimeTicks service_worker_ready_time() const {
-    return service_worker_ready_time_;
-  }
-  bool response_is_in_cache_storage() const {
-    return response_is_in_cache_storage_;
-  }
-  const std::string& response_cache_storage_cache_name() const {
-    return response_cache_storage_cache_name_;
-  }
-  bool did_navigation_preload() const { return did_navigation_preload_; }
-
- private:
-  ServiceWorkerResponseInfo();
-
-  bool was_fetched_via_service_worker_ = false;
-  bool was_fallback_required_ = false;
-  std::vector<GURL> url_list_via_service_worker_;
-  network::mojom::FetchResponseType response_type_via_service_worker_ =
-      network::mojom::FetchResponseType::kDefault;
-  base::TimeTicks service_worker_start_time_;
-  base::TimeTicks service_worker_ready_time_;
-  bool response_is_in_cache_storage_ = false;
-  std::string response_cache_storage_cache_name_;
-  ServiceWorkerHeaderList cors_exposed_header_names_;
-  bool did_navigation_preload_ = false;
-
-  static int user_data_key_;  // Only address is used.
-};
-
-}  // namespace content
-
-#endif  // CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_RESPONSE_INFO_H_
diff --git a/services/network/public/cpp/resource_response_info.h b/services/network/public/cpp/resource_response_info.h
index 344a6fa..d11e644 100644
--- a/services/network/public/cpp/resource_response_info.h
+++ b/services/network/public/cpp/resource_response_info.h
@@ -111,7 +111,12 @@
   // The proxy server used for this request, if any.
   net::ProxyServer proxy_server;
 
-  // True if the response was fetched by a ServiceWorker.
+  // True if a service worker responded to the request. If the service worker
+  // received a fetch event and did not call respondWith(), or was bypassed due
+  // to absence of a fetch event handler, this function typically returns false
+  // but returns true if "fallback to renderer" was required (however in this
+  // case the response is not an actual resource and the request will be
+  // reissued by the renderer).
   bool was_fetched_via_service_worker;
 
   // True when a request whose mode is |CORS| or |CORS-with-forced-preflight|
@@ -120,8 +125,14 @@
   // considering the CORS preflight logic.
   bool was_fallback_required_by_service_worker;
 
-  // The URL list of the response which was served by the ServiceWorker. See
-  // ServiceWorkerResponseInfo::url_list_via_service_worker().
+  // The URL list of the Response object the service worker passed to
+  // respondWith() to create this response. For example, if the service worker
+  // calls respondWith(fetch('http://example.com/a')) and http://example.com/a
+  // redirects to http://example.net/b which redirects to http://example.org/c,
+  // the URL list is the vector <"http://example.com/a", "http://example.net/b",
+  // "http://example.org/c">. This is empty if the response was programmatically
+  // generated as in respondWith(new Response()). It is also empty if a service
+  // worker did not respond to the request or did not call respondWith().
   std::vector<GURL> url_list_via_service_worker;
 
   // https://fetch.spec.whatwg.org/#concept-response-type
diff --git a/third_party/blink/public/platform/web_url_response.h b/third_party/blink/public/platform/web_url_response.h
index 83d7885..c7609e6 100644
--- a/third_party/blink/public/platform/web_url_response.h
+++ b/third_party/blink/public/platform/web_url_response.h
@@ -232,13 +232,11 @@
   BLINK_PLATFORM_EXPORT void SetWasFetchedViaSPDY(bool);
 
   // Flag whether this request was loaded via a ServiceWorker. See
-  // ServiceWorkerResponseInfo::was_fetched_via_service_worker() for details.
+  // network::ResourceResponseInfo::was_fetched_via_service_worker for details.
   BLINK_PLATFORM_EXPORT bool WasFetchedViaServiceWorker() const;
   BLINK_PLATFORM_EXPORT void SetWasFetchedViaServiceWorker(bool);
 
-  // Flag whether the fallback request with skip service worker flag was
-  // required. See ServiceWorkerResponseInfo::was_fallback_required() for
-  // details.
+  // See network::ResourceResponseInfo::was_fallback_required_by_service_worker.
   BLINK_PLATFORM_EXPORT void SetWasFallbackRequiredByServiceWorker(bool);
 
   // https://fetch.spec.whatwg.org/#concept-response-type
@@ -246,8 +244,8 @@
   BLINK_PLATFORM_EXPORT network::mojom::FetchResponseType GetType() const;
 
   // The URL list of the Response object the ServiceWorker passed to
-  // respondWith(). See ServiceWorkerResponseInfo::url_list_via_service_worker()
-  // for details.
+  // respondWith(). See
+  // network::ResourceResponseInfo::url_list_via_service_worker for details.
   BLINK_PLATFORM_EXPORT void SetUrlListViaServiceWorker(
       const WebVector<WebURL>&);
   // Returns true if the URL list is not empty.
@@ -264,8 +262,7 @@
       const WebVector<WebString>&);
 
   // Whether service worker navigation preload occurred.
-  // See ServiceWorkerResponseInfo::did_navigation_preload() for
-  // details.
+  // See network::ResourceResponseInfo::did_navigation_preload for details.
   BLINK_PLATFORM_EXPORT void SetDidServiceWorkerNavigationPreload(bool);
 
   // Remote IP address of the socket which fetched this resource.
diff --git a/third_party/blink/renderer/platform/loader/fetch/resource_response.h b/third_party/blink/renderer/platform/loader/fetch/resource_response.h
index 161648c..7059a04 100644
--- a/third_party/blink/renderer/platform/loader/fetch/resource_response.h
+++ b/third_party/blink/renderer/platform/loader/fetch/resource_response.h
@@ -296,7 +296,7 @@
   bool WasFetchedViaSPDY() const { return was_fetched_via_spdy_; }
   void SetWasFetchedViaSPDY(bool value) { was_fetched_via_spdy_ = value; }
 
-  // See ServiceWorkerResponseInfo::was_fetched_via_service_worker.
+  // See network::ResourceResponseInfo::was_fetched_via_service_worker.
   bool WasFetchedViaServiceWorker() const {
     return was_fetched_via_service_worker_;
   }
@@ -304,7 +304,7 @@
     was_fetched_via_service_worker_ = value;
   }
 
-  // See ServiceWorkerResponseInfo::was_fallback_required.
+  // See network::ResourceResponseInfo::was_fallback_required_by_service_worker.
   bool WasFallbackRequiredByServiceWorker() const {
     return was_fallback_required_by_service_worker_;
   }
@@ -325,7 +325,7 @@
     return network::cors::IsCorsCrossOriginResponseType(response_type_);
   }
 
-  // See ServiceWorkerResponseInfo::url_list_via_service_worker.
+  // See network::ResourceResponseInfo::url_list_via_service_worker.
   const Vector<KURL>& UrlListViaServiceWorker() const {
     return url_list_via_service_worker_;
   }