Prerender: Confirm ServiceWorkers are invoked for NoState Prefetch

content/browser/service_worker/service_worker_context_wrapper.h

 // Copyright 2013 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_CONTEXT_WRAPPER_H_
 #define CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_CONTEXT_WRAPPER_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 90 matching lines @@
   void DeleteForOrigin(const GURL& origin,
                        const ResultCallback& callback) override;
   void CheckHasServiceWorker(
       const GURL& url,
       const GURL& other_url,
       const CheckHasServiceWorkerCallback& callback) override;
   void CountExternalRequestsForTest(
       const GURL& url,
       const CountExternalRequestsCallback& callback) override;
   void StopAllServiceWorkersForOrigin(const GURL& origin) override;
+  void StopAllServiceWorkersForOriginWithCallback(

[falken, 2017/01/06 16:17:55]

These functions are a bit misleading since ServiceWorkerVersion::StopWorker()
doesn't necessarily mean the worker is really stopped for good. It could be
restarted soon after the callback is invoked in
ServiceWorkerVersion::OnStoppedInternal (see the |should_restart| thing). I'd
rather not have a function with a callback that seems to indicate "all these
workers are really stopped" since we don't yet guarantee that.

[mattcary, 2017/01/09 14:45:42]

LMK if the new big-hammer seems better. Also if it should be marked as for
testing or whatever.

+      const GURL& origin,
+      const base::Closure& callback) override;
   void ClearAllServiceWorkersForTest(const base::Closure& callback) override;
   void StartServiceWorkerForNavigationHint(
       const GURL& document_url,
       blink::WebNavigationHintType type,
       int render_process_id,
       const ResultCallback& callback) override;
   bool StartingExternalRequest(int64_t service_worker_version_id,
                                const std::string& request_uuid) override;
   bool FinishedExternalRequest(int64_t service_worker_version_id,
                                const std::string& request_uuid) override;
@@ skipping 179 matching lines @@
 
   // Must be touched on the UI thread.
   std::map<int, int> navigation_hint_task_count_per_process_;
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerContextWrapper);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_CONTEXT_WRAPPER_H_