[extension SW] Support lazy events from extension service workers.

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 40 matching lines @@
 
   using CheckHasServiceWorkerCallback =
       base::Callback<void(ServiceWorkerCapability capability)>;
 
   using CountExternalRequestsCallback =
       base::Callback<void(size_t external_request_count)>;
 
   using StartServiceWorkerForNavigationHintCallback =
       base::Callback<void(StartServiceWorkerForNavigationHintResult result)>;
 
+  using GetWorkerInfoCallback =
+      base::OnceCallback<void(int process_id, int thread_id)>;
+
   // Registers the header name which should not be passed to the ServiceWorker.
   // Must be called from the IO thread.
   CONTENT_EXPORT static void AddExcludedHeadersForFetchEvent(
       const std::set<std::string>& header_names);
 
   // Returns true if the header name should not be passed to the ServiceWorker.
   // Must be called from the IO thread.
   static bool IsExcludedHeaderNameForFetchEvent(const std::string& header_name);
 
   // Returns true if |url| is within the service worker |scope|.
@@ skipping 29 matching lines @@
   // shut the worker down while embedder is expecting the worker to be kept
   // alive.
   //
   // Must be called from the IO thread. Returns whether or not changing the ref
   // count succeeded.
   virtual bool StartingExternalRequest(int64_t service_worker_version_id,
                                        const std::string& request_uuid) = 0;
   virtual bool FinishedExternalRequest(int64_t service_worker_version_id,
                                        const std::string& request_uuid) = 0;
 
+  // Starts a previously registered worker at |origin| and returns running

[falken, 2017/06/22 03:35:57]

Starts the active worker of the registration whose scope is |pattern| and...

[lazyboy, 2017/06/23 02:18:30]

Done.

+  // worker info via |info_callback|.
+  // |info_callback| contains worker render process id and worker thread id.

[falken, 2017/06/22 03:35:57]

Is "contains" the conventional wording for a callback? I would have said
"|info_callback| is passed the worker's render process..."

[lazyboy, 2017/06/23 02:18:30]

That's better, thanks.
Done.

+  //
+  // Must be called on IO thread.
+  virtual void GetWorkerInfoAfterStartWorker(
+      const GURL& origin,

[falken, 2017/06/22 03:35:57]

|origin| looks weird to me because an origin can have multiple registrations,
and a registration has multiple workers.

What is being passed here? Since function uses FindReadyRegistrationForPattern
you must pass in a pattern here that is identical to the registration's pattern
(i.e., it doesn't do a prefix match like FindReadyRegistrationForDocument uses).

So I think this should be called
GetWorkerInfoAfterStartActiveWorkerForPattern(const GURL& pattern, ...)

or maybe just StartActiveWorkerForPattern.

[lazyboy, 2017/06/23 02:18:30]

It is origin + scope,
For example,
chrome-extension://pdbihmnjpcgicblhmommoeoegalabckg/
where the extension pdbihmnjpcgicblhmommoeoegalabckg registered a service worker
on the scope '/'.

I've changed the references to origin in the CL to pattern.

 Since function uses FindReadyRegistrationForPattern
Changed to StartActiveWorkerForPattern

+      GetWorkerInfoCallback info_callback,
+      base::OnceClosure failure_callback) = 0;
+
   // Equivalent to calling navigator.serviceWorker.unregister(pattern) from a
   // renderer, except that |pattern| is an absolute URL instead of relative to
   // some current origin.  |callback| is passed true when the JS promise is
   // fulfilled or false when the JS promise is rejected.
   //
   // Unregistration can fail if:
   //  * No Service Worker was registered for |pattern|.
   //  * Something unexpected goes wrong, like a renderer crash.
   //
   // This function can be called from any thread, but the callback will always
@@ skipping 54 matching lines @@
       const StartServiceWorkerForNavigationHintCallback& callback) = 0;
 
  protected:
   ServiceWorkerContext() {}
   virtual ~ServiceWorkerContext() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_SERVICE_WORKER_CONTEXT_H_