Prerender: Confirm ServiceWorkers are invoked for NoState Prefetch

content/public/browser/service_worker_context.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_SERVICE_WORKER_CONTEXT_H_
 #define CONTENT_PUBLIC_BROWSER_SERVICE_WORKER_CONTEXT_H_
 
 #include <set>
 #include <string>
 
@@ skipping 107 matching lines @@
   // specified |origin| via |callback|.
   virtual void CountExternalRequestsForTest(
       const GURL& origin,
       const CountExternalRequestsCallback& callback) = 0;
 
   // Stops all running workers on the given |origin|.
   //
   // This function can be called from any thread.
   virtual void StopAllServiceWorkersForOrigin(const GURL& origin) = 0;
 
+  // As above, but |callback| is exectued when a running worker is stopped. If

[pasko, 2017/01/02 18:15:36]

nit: s/exectued/executed/

[mattcary, 2017/01/03 09:00:03]

Done.

+  // there is no running worker, |callback| will never be executed.

[pasko, 2017/01/02 18:15:36]

My personal preference would be with _always_ invoking the callback, with status
depending on whether there was a SW running.

Not my call though, up to content owners.

[mattcary, 2017/01/03 09:00:03]

Given its current use in a test these semantics work fine. 

Adding more information for speculative future use makes me nervous for several
reasons: I was not really interested in widening the API by adding a public
status enumeration (or exposing the existing internal status enum). Also, there
seems to be a range of behavior depending on the state of
ServiceWorkerVersion---exposing a status will probably necessarily leak the
implementation (eg, a ServiceWorkerVersion that exists but which is already
stopped, etc).

+  virtual void StopAllServiceWorkersForOriginWithCallback(
+      const GURL& origin,
+      const base::Closure& callback) = 0;
+
   // Stops all running service workers and unregisters all service worker
   // registrations. This method is used in LayoutTests to make sure that the
   // existing service worker will not affect the succeeding tests.
   //
   // This function can be called from any thread, but the callback will always
   // be called on the UI thread.
   virtual void ClearAllServiceWorkersForTest(const base::Closure& callback) = 0;
 
   // Starts a Service Worker for |document_url| for a navigation hint in the
   // specified render process |render_process_id|. Must be called from the UI
   // thread. The |callback| will always be called on the UI thread.
   // This method can fail if:
   //  * No Service Worker was registered for |document_url|.
   //  * The specified render process is not suitable for loading |document_url|.
   virtual void StartServiceWorkerForNavigationHint(
       const GURL& document_url,
       blink::WebNavigationHintType type,
       int render_process_id,
       const ResultCallback& callback) = 0;
 
  protected:
   ServiceWorkerContext() {}
   virtual ~ServiceWorkerContext() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_SERVICE_WORKER_CONTEXT_H_