Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1802)

Unified Diff: chrome/common/extensions/manifest_unittest.cc

Issue 13971005: Basic multi-module support (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix copyright for presubmit Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/extensions/manifest_unittest.cc
diff --git a/chrome/common/extensions/manifest_unittest.cc b/chrome/common/extensions/manifest_unittest.cc
index 6805022084232b0858a18ae6e67ef2b6bd3dcebf..8845ae017db30bc4a3adafc405bf9917f6d46e9f 100644
--- a/chrome/common/extensions/manifest_unittest.cc
+++ b/chrome/common/extensions/manifest_unittest.cc
@@ -36,6 +36,8 @@ class ManifestTest : public testing::Test {
EXPECT_EQ(type == Manifest::TYPE_LEGACY_PACKAGED_APP,
manifest->is_legacy_packaged_app());
EXPECT_EQ(type == Manifest::TYPE_HOSTED_APP, manifest->is_hosted_app());
+ EXPECT_EQ(type == Manifest::TYPE_SHARED_MODULE,
+ manifest->is_shared_module());
}
// Helper function that replaces the Manifest held by |manifest| with a copy
@@ -138,6 +140,13 @@ TEST_F(ManifestTest, ExtensionTypes) {
MutateManifest(
&manifest, keys::kTheme, NULL);
+ // Shared module.
+ MutateManifest(
+ &manifest, keys::kExport, new base::DictionaryValue());
+ AssertType(manifest.get(), Manifest::TYPE_SHARED_MODULE);
+ MutateManifest(
+ &manifest, keys::kExport, NULL);
+
// Packaged app.
MutateManifest(
&manifest, keys::kApp, new base::DictionaryValue());

Powered by Google App Engine
This is Rietveld 408576698