Begin to enable extension APIs in Extension Service Worker.

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 11 matching lines @@
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/content_switches.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 "extensions/browser/extension_host.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/process_manager.h"
+#include "extensions/common/features/feature_util.h"
 #include "extensions/test/background_page_watcher.h"
 #include "extensions/test/extension_test_message_listener.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
 
 namespace extensions {
 
 namespace {
 
 // Pass into ServiceWorkerTest::StartTestFromBackgroundPage to indicate that
 // registration is expected to succeed.
@@ skipping 31 matching lines @@
     if (message_loop_runner_)
       message_loop_runner_->Quit();
   }
 
   bool load_stop_observed_;
   scoped_refptr<content::MessageLoopRunner> message_loop_runner_;
 
   DISALLOW_COPY_AND_ASSIGN(WebContentsLoadStopObserver);
 };
 
+class ScopedExtensionAPIInServiceWorkerEnabler {
+ public:
+  ScopedExtensionAPIInServiceWorkerEnabler()
+      : current_value_(
+            feature_util::ExtensionAPIEnabledInExtensionServiceWorkers()) {
+    feature_util::SetExtensionAPIEnabledInExtensionServiceWorkers(true);
+  }
+  ~ScopedExtensionAPIInServiceWorkerEnabler() {
+    feature_util::SetExtensionAPIEnabledInExtensionServiceWorkers(
+        current_value_);
+  }
+
+ private:
+  bool current_value_;
+  DISALLOW_COPY_AND_ASSIGN(ScopedExtensionAPIInServiceWorkerEnabler);
+};
+
 }  // namespace
 
 class ServiceWorkerTest : public ExtensionApiTest {
  public:
   ServiceWorkerTest() : current_channel_(version_info::Channel::DEV) {}
 
   ~ServiceWorkerTest() override {}
 
  protected:
   // Returns the ProcessManager for the test's profile.
@@ skipping 504 matching lines @@
 IN_PROC_BROWSER_TEST_F(ServiceWorkerTest, NotificationAPI) {
   EXPECT_TRUE(RunExtensionSubtest("service_worker/notifications/has_permission",
                                   "page.html"));
 }
 
 IN_PROC_BROWSER_TEST_F(ServiceWorkerTest, WebAccessibleResourcesFetch) {
   EXPECT_TRUE(RunExtensionSubtest(
       "service_worker/web_accessible_resources/fetch/", "page.html"));
 }
 
+IN_PROC_BROWSER_TEST_F(ServiceWorkerTest, TabsCreate) {
+  ScopedExtensionAPIInServiceWorkerEnabler enabler;
+  const Extension* extension = LoadExtensionWithFlags(
+      test_data_dir_.AppendASCII("service_worker/tabs_create"), kFlagNone);
+  ASSERT_TRUE(extension);
+  ui_test_utils::NavigateToURL(browser(),
+                               extension->GetResourceURL("page.html"));
+  content::WebContents* web_contents =
+      browser()->tab_strip_model()->GetActiveWebContents();
+
+  int starting_tab_count = browser()->tab_strip_model()->count();
+  std::string result;
+  ASSERT_TRUE(content::ExecuteScriptAndExtractString(
+      web_contents, "window.runServiceWorker()", &result));

[Devlin, 2016/04/14 22:34:52]

Shouldn't we check |result|?

[lazyboy, 2016/04/15 00:57:47]

Wow, this got deleted somehow.
Fixed.

+  EXPECT_EQ(starting_tab_count + 1, browser()->tab_strip_model()->count());
+
+  // Check extension shutdown path.
+  UnloadExtension(extension->id());
+  EXPECT_EQ(starting_tab_count, browser()->tab_strip_model()->count());
+}
+
 // This test loads a web page that has an iframe pointing to a
 // chrome-extension:// URL. The URL is listed in the extension's
 // web_accessible_resources. Initially the iframe is served from the extension's
 // resource file. After verifying that, we register a Service Worker that
 // controls the extension. Further requests to the same resource as before
 // should now be served by the Service Worker.
 // This test also verifies that if the requested resource exists in the manifest
 // but is not present in the extension directory, the Service Worker can still
 // serve the resource file.
 IN_PROC_BROWSER_TEST_F(ServiceWorkerTest, WebAccessibleResourcesIframeSrc) {
@@ skipping 122 matching lines @@
   push_message_listener.set_failure_message("FAIL");
   gcm::IncomingMessage message;
   message.sender_id = "1234567890";
   message.raw_data = "testdata";
   message.decrypted = true;
   push_service()->OnMessage(app_identifier.app_id(), message);
   EXPECT_TRUE(push_message_listener.WaitUntilSatisfied());
 }
 
 }  // namespace extensions