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

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

Issue 8662008: Implement chrome.systemPrivate.getIncognitoModeAvailability extension API function (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merged with ToT Created 9 years 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/extension_permission_set_unittest.cc
diff --git a/chrome/common/extensions/extension_permission_set_unittest.cc b/chrome/common/extensions/extension_permission_set_unittest.cc
index 8eb1b7895d35ff2ba0fe382351f24badb1ee983e..73631e26a639523556eb0cb6393b519416d8ef5b 100644
--- a/chrome/common/extensions/extension_permission_set_unittest.cc
+++ b/chrome/common/extensions/extension_permission_set_unittest.cc
@@ -199,6 +199,7 @@ TEST(ExtensionAPIPermissionTest, ComponentOnlyPermissions) {
private_perms.insert(ExtensionAPIPermission::kFileBrowserPrivate);
private_perms.insert(ExtensionAPIPermission::kMediaPlayerPrivate);
private_perms.insert(ExtensionAPIPermission::kMetricsPrivate);
+ private_perms.insert(ExtensionAPIPermission::kSystemPrivate);
private_perms.insert(ExtensionAPIPermission::kWebstorePrivate);
ExtensionAPIPermissionSet perms = info->GetAll();
@@ -210,7 +211,7 @@ TEST(ExtensionAPIPermissionTest, ComponentOnlyPermissions) {
info->GetByID(*i)->is_component_only());
}
- EXPECT_EQ(6, count);
+ EXPECT_EQ(7, count);
}
TEST(ExtensionPermissionSetTest, EffectiveHostPermissions) {
@@ -663,15 +664,16 @@ TEST(ExtensionPermissionSetTest, PermissionMessages) {
skip.insert(ExtensionAPIPermission::kExperimental);
// These are private.
- skip.insert(ExtensionAPIPermission::kWebstorePrivate);
+ skip.insert(ExtensionAPIPermission::kChromeAuthPrivate);
+ skip.insert(ExtensionAPIPermission::kChromeosInfoPrivate);
+ skip.insert(ExtensionAPIPermission::kChromePrivate);
skip.insert(ExtensionAPIPermission::kFileBrowserPrivate);
+ skip.insert(ExtensionAPIPermission::kInputMethodPrivate);
skip.insert(ExtensionAPIPermission::kMediaPlayerPrivate);
skip.insert(ExtensionAPIPermission::kMetricsPrivate);
- skip.insert(ExtensionAPIPermission::kChromeAuthPrivate);
- skip.insert(ExtensionAPIPermission::kChromePrivate);
- skip.insert(ExtensionAPIPermission::kChromeosInfoPrivate);
+ skip.insert(ExtensionAPIPermission::kSystemPrivate);
skip.insert(ExtensionAPIPermission::kWebSocketProxyPrivate);
- skip.insert(ExtensionAPIPermission::kInputMethodPrivate);
+ skip.insert(ExtensionAPIPermission::kWebstorePrivate);
// Warned as part of host permissions.
skip.insert(ExtensionAPIPermission::kDevtools);
« no previous file with comments | « chrome/common/extensions/extension_permission_set.cc ('k') | chrome/test/data/extensions/api_test/system/manifest.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698