Implement createfailed event for <extensionoptions>

chrome/browser/guest_view/extension_options/extension_options_apitest.cc

 // 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.
 
+#include "base/files/file_path.h"
 #include "chrome/browser/extensions/extension_apitest.h"
+#include "chrome/test/base/ui_test_utils.h"
 #include "extensions/common/feature_switch.h"
 #include "extensions/common/switches.h"
 
 using extensions::Extension;
 using extensions::FeatureSwitch;
 
 class ExtensionOptionsApiTest : public ExtensionApiTest {
  private:
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+    enable_options_.reset(new FeatureSwitch::ScopedOverride(
+        FeatureSwitch::embedded_extension_options(), true));
     // Need to add a command line flag as well as a FeatureSwitch because the
     // FeatureSwitch is not copied over to the renderer process from the
     // browser process.
     command_line->AppendSwitch(
         extensions::switches::kEnableEmbeddedExtensionOptions);
     ExtensionApiTest::SetUpCommandLine(command_line);

[not at google - send to devlin, 2014/08/04 19:12:03]

ideally call the superclass SetUpCommandLine first, since that's the way that
construction order goes.

[ericzeng, 2014/08/04 21:15:33]

Done.

   }
+
+  scoped_ptr<FeatureSwitch::ScopedOverride> enable_options_;
 };
 
 IN_PROC_BROWSER_TEST_F(ExtensionOptionsApiTest, ExtensionCanEmbedOwnOptions) {
+  base::FilePath extension_dir =
+      test_data_dir_.AppendASCII("extension_options").AppendASCII("embed_self");
+  ASSERT_TRUE(InstallExtension(extension_dir, 1));
+  ASSERT_TRUE(RunExtensionSubtest("extension_options/embed_self", "test.html"));
+}
+
+IN_PROC_BROWSER_TEST_F(ExtensionOptionsApiTest,
+                       ShouldNotEmbedOtherExtensionsOptions) {
   FeatureSwitch::ScopedOverride enable_options(
       FeatureSwitch::embedded_extension_options(), true);
+  base::FilePath dir = test_data_dir_.AppendASCII("extension_options")
+                           .AppendASCII("embed_other");
 
-  const Extension* extension = InstallExtension(
-      test_data_dir_.AppendASCII("extension_options").AppendASCII("embed_self"),
-      1);
-  ASSERT_TRUE(extension);
-  ASSERT_TRUE(RunExtensionSubtest("extension_options/embed_self", "test.html"));
+  const Extension* embedder = InstallExtension(dir.AppendASCII("embedder"), 1);
+  const Extension* embedded = InstallExtension(dir.AppendASCII("embedded"), 1);
+
+  ASSERT_TRUE(embedder);
+  ASSERT_TRUE(embedded);
+
+  std::stringstream script;

[not at google - send to devlin, 2014/08/04 19:12:03]

don't use stringstream. It's just as easy to construct this like

std::string script = base::StringPrintf(
    "chrome.storage.local.set({embeddedId: '%s}, function() {"
    "  window.domAutomationController.send('done injecting');"
    "});", embedded->id());

or thereabouts.

[ericzeng, 2014/08/04 21:15:33]

Done.

+  script << "chrome.storage.local.set("
+         << "{'embeddedId': '" << embedded->id() << "'}, function() {"
+         << "window.domAutomationController.send('done injecting');"
+         << "});";
+
+  ExecuteScriptInBackgroundPage(embedder->id(), script.str());
+  ResultCatcher catcher;
+  ui_test_utils::NavigateToURL(browser(),
+                               embedder->GetResourceURL("test.html"));
+  ASSERT_TRUE(catcher.GetNextResult());
 }