Index: chrome/common/extensions/extension_manifests_unittest.cc |
diff --git a/chrome/common/extensions/extension_manifests_unittest.cc b/chrome/common/extensions/extension_manifests_unittest.cc |
index 3a891666537cc2a88269429db57fb0fea32272c3..446ff31bfd127703abec59ef9982279682e404cb 100644 |
--- a/chrome/common/extensions/extension_manifests_unittest.cc |
+++ b/chrome/common/extensions/extension_manifests_unittest.cc |
@@ -294,7 +294,7 @@ TEST_F(ExtensionManifestTest, OptionsPageInApps) { |
errors::kInvalidOptionsPageExpectUrlInPackage); |
} |
-TEST_F(ExtensionManifestTest, DisallowExtensionPermissions) { |
+TEST_F(ExtensionManifestTest, AllowUnrecognizedPermissions) { |
std::string error; |
scoped_ptr<DictionaryValue> manifest( |
LoadManifestFile("valid_app.json", &error)); |
@@ -308,13 +308,11 @@ TEST_F(ExtensionManifestTest, DisallowExtensionPermissions) { |
permissions->Clear(); |
permissions->Append(p); |
std::string message_name = base::StringPrintf("permission-%s", name); |
- if (Extension::IsHostedAppPermission(name)) { |
- scoped_refptr<Extension> extension; |
- extension = LoadAndExpectSuccess(manifest.get(), message_name); |
- } else { |
- LoadAndExpectError(manifest.get(), message_name, |
- errors::kInvalidPermission); |
- } |
+ |
+ // Extensions are allowed to contain unrecognized API permissions, |
+ // so there shouldn't be any errors. |
+ scoped_refptr<Extension> extension; |
+ extension = LoadAndExpectSuccess(manifest.get(), message_name); |
} |
} |