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

chrome/browser/extensions/service_worker_apitest.cc

Index: chrome/browser/extensions/service_worker_apitest.cc
diff --git a/chrome/browser/extensions/service_worker_apitest.cc b/chrome/browser/extensions/service_worker_apitest.cc
index 54dc7462cf1f94b9dd4d7bf0aad1ec034f52eea8..73d34488efddd7251b8231eb636ea9867aa826d1 100644
--- a/chrome/browser/extensions/service_worker_apitest.cc
+++ b/chrome/browser/extensions/service_worker_apitest.cc
@@ -36,6 +36,7 @@
 #include "content/public/common/page_type.h"
 #include "content/public/test/background_sync_test_util.h"
 #include "content/public/test/browser_test_utils.h"
+#include "content/public/test/service_worker_test_helpers.h"
 #include "extensions/browser/extension_host.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/process_manager.h"
@@ -43,6 +44,7 @@
 #include "extensions/test/extension_test_message_listener.h"
 #include "net/dns/mock_host_resolver.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
+#include "url/url_constants.h"
 
 namespace extensions {
 
@@ -682,6 +684,42 @@ IN_PROC_BROWSER_TEST_F(ServiceWorkerTest, Events) {
   ASSERT_EQ("chrome.tabs.onUpdated callback", result);
 }
 
+IN_PROC_BROWSER_TEST_F(ServiceWorkerTest, EventsToStoppedWorker) {
+  // Extensions APIs from SW are only enabled on trunk.
+  ScopedCurrentChannel current_channel_override(version_info::Channel::UNKNOWN);
+  const Extension* extension = LoadExtensionWithFlags(
+      test_data_dir_.AppendASCII("service_worker/events_to_stopped_worker"),
+      kFlagNone);
+  ASSERT_TRUE(extension);
+  ui_test_utils::NavigateToURL(browser(),
+                               extension->GetResourceURL("page.html"));
+  content::WebContents* web_contents =
+      browser()->tab_strip_model()->GetActiveWebContents();
+  {
+    std::string result;
+    ASSERT_TRUE(content::ExecuteScriptAndExtractString(
+        web_contents, "window.runServiceWorker()", &result));
+    ASSERT_EQ("ready", result);
+
+    {

[Devlin, 2017/06/20 20:31:19]

nit: do these scopes need to be nested?

[lazyboy, 2017/06/22 02:46:56]

Done.

+      base::RunLoop stop_called_run_loop;

[Devlin, 2017/06/20 20:31:19]

nit: this is a bit weird, since the run loop isn't to stop a run loop.  Maybe
something like "run_until_worker_stopped"?  Or just "run_loop" is fine too. :)

[lazyboy, 2017/06/22 02:46:56]

Done.

+      content::StoragePartition* storage_partition =
+          content::BrowserContext::GetDefaultStoragePartition(
+              browser()->profile());
+      content::StopServiceWorkerAtOrigin(
+          storage_partition->GetServiceWorkerContext(),
+          // The service worker is registered at the top level scope.
+          extension->url(), stop_called_run_loop.QuitClosure());
+      stop_called_run_loop.Run();
+    }
+  }
+
+  std::string result;
+  ASSERT_TRUE(content::ExecuteScriptAndExtractString(
+      web_contents, "window.createTabThenUpdate()", &result));
+  ASSERT_EQ("chrome.tabs.onUpdated callback", result);
+}
+
 // Tests that worker ref count increments while extension API function is
 // active.