Prerender: Confirm ServiceWorkers are invoked for NoState Prefetch

content/browser/service_worker/service_worker_context_wrapper.h

Index: content/browser/service_worker/service_worker_context_wrapper.h
diff --git a/content/browser/service_worker/service_worker_context_wrapper.h b/content/browser/service_worker/service_worker_context_wrapper.h
index cadaaada440c9c5049d60d9aeccf3319ece2c153..2f288d983d9b27fb16e985c74db8f8c648ebf078 100644
--- a/content/browser/service_worker/service_worker_context_wrapper.h
+++ b/content/browser/service_worker/service_worker_context_wrapper.h
@@ -108,6 +108,9 @@ class CONTENT_EXPORT ServiceWorkerContextWrapper
       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,