Basic multi-module support

chrome/browser/extensions/shared_module_apitest.cc

+// Copyright (c) 2013 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 "chrome/browser/extensions/extension_apitest.h"
+
+using extensions::Extension;
+
+IN_PROC_BROWSER_TEST_F(ExtensionApiTest, SharedModule) {
+  // import_pass depends on this shared module.
+  // NB: We use LoadExtension instead of InstallExtension here so the public-key
+  // in 'shared' is used to generate the extension ID so it can be imported
+  // correctly.  We use InstallExtension otherwise so the loads happen through
+  // the CRX installer which validates imports.
+  ASSERT_TRUE(LoadExtension(
+      test_data_dir_.AppendASCII("shared_module").AppendASCII("shared")));
+
+  ResultCatcher catcher;
+  EXPECT_TRUE(InstallExtension(
+      test_data_dir_.AppendASCII("shared_module").AppendASCII("import_pass"),
+      1));
+  EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();

[asargent_no_longer_on_chrome, 2013/04/27 00:13:50]

I think you might be able to slightly simplify this code (drop one of the Load
calls, and not need your own ResultCatcher) if you used one of the helper
methods you inherit like  ExtensionApiTest::RunExtensionTest or
ExtensionApiTest::RunPageTest.

[elijahtaylor1, 2013/04/27 00:26:10]

Good call, Done

+
+  EXPECT_FALSE(InstallExtension(
+      test_data_dir_.AppendASCII("shared_module")
+          .AppendASCII("import_wrong_version"), 0));
+  EXPECT_FALSE(InstallExtension(
+      test_data_dir_.AppendASCII("shared_module")
+          .AppendASCII("import_non_existent"), 0));
+}