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

chrome/browser/extensions/service_worker_apitest.cc

 // Copyright 2015 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.
 
 #include <stdint.h>
 
 #include "base/bind_helpers.h"
 #include "base/macros.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 18 matching lines @@
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/service_worker_context.h"
 #include "content/public/browser/storage_partition.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/common/origin_util.h"
 #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"
 #include "extensions/test/background_page_watcher.h"
 #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 {
 
 namespace {
 
 // Pass into ServiceWorkerTest::StartTestFromBackgroundPage to indicate that
 // registration is expected to succeed.
 std::string* const kExpectSuccess = nullptr;
 
 void DoNothingWithBool(bool b) {}
@@ skipping 619 matching lines @@
   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.runEventTest()", &result));
   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.
 
 // Flaky on Linux and ChromeOS, https://crbug.com/702126
 #if defined(OS_LINUX)
 #define MAYBE_WorkerRefCount DISABLED_WorkerRefCount
 #else
 #define MAYBE_WorkerRefCount WorkerRefCount
 #endif
 IN_PROC_BROWSER_TEST_F(ServiceWorkerTest, MAYBE_WorkerRefCount) {
@@ skipping 203 matching lines @@
   message.sender_id = "1234567890";
   message.raw_data = "testdata";
   message.decrypted = true;
   push_service()->SetMessageCallbackForTesting(run_loop.QuitClosure());
   push_service()->OnMessage(app_identifier.app_id(), message);
   EXPECT_TRUE(push_message_listener.WaitUntilSatisfied());
   run_loop.Run();  // Wait until the message is handled by push service.
 }
 
 }  // namespace extensions