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

Side by Side Diff: chrome/common/extensions/extension_permission_set_unittest.cc

Issue 8747003: Add private system extension API to get system update status (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Codereview 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/common/extensions/extension_permission_set.h" 5 #include "chrome/common/extensions/extension_permission_set.h"
6 6
7 #include "base/json/json_value_serializer.h" 7 #include "base/json/json_value_serializer.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/path_service.h" 9 #include "base/path_service.h"
10 #include "base/utf_string_conversions.h" 10 #include "base/utf_string_conversions.h"
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 EXPECT_EQ(hosted_perms.count(*i) > 0, info->GetByID(*i)->is_hosted_app()); 161 EXPECT_EQ(hosted_perms.count(*i) > 0, info->GetByID(*i)->is_hosted_app());
162 } 162 }
163 163
164 EXPECT_EQ(hosted_perms.size(), count); 164 EXPECT_EQ(hosted_perms.size(), count);
165 EXPECT_EQ(hosted_perms.size(), info->get_hosted_app_permission_count()); 165 EXPECT_EQ(hosted_perms.size(), info->get_hosted_app_permission_count());
166 } 166 }
167 167
168 TEST(ExtensionAPIPermissionTest, ComponentOnlyPermissions) { 168 TEST(ExtensionAPIPermissionTest, ComponentOnlyPermissions) {
169 ExtensionPermissionsInfo* info = ExtensionPermissionsInfo::GetInstance(); 169 ExtensionPermissionsInfo* info = ExtensionPermissionsInfo::GetInstance();
170 ExtensionAPIPermissionSet private_perms; 170 ExtensionAPIPermissionSet private_perms;
171 private_perms.insert(ExtensionAPIPermission::kAccessibilityPrivate);
171 private_perms.insert(ExtensionAPIPermission::kChromeAuthPrivate); 172 private_perms.insert(ExtensionAPIPermission::kChromeAuthPrivate);
172 private_perms.insert(ExtensionAPIPermission::kChromeosInfoPrivate); 173 private_perms.insert(ExtensionAPIPermission::kChromeosInfoPrivate);
173 private_perms.insert(ExtensionAPIPermission::kFileBrowserPrivate); 174 private_perms.insert(ExtensionAPIPermission::kFileBrowserPrivate);
174 private_perms.insert(ExtensionAPIPermission::kMediaPlayerPrivate); 175 private_perms.insert(ExtensionAPIPermission::kMediaPlayerPrivate);
175 private_perms.insert(ExtensionAPIPermission::kMetricsPrivate); 176 private_perms.insert(ExtensionAPIPermission::kMetricsPrivate);
176 private_perms.insert(ExtensionAPIPermission::kWebstorePrivate); 177 private_perms.insert(ExtensionAPIPermission::kWebstorePrivate);
177 178
178 ExtensionAPIPermissionSet perms = info->GetAll(); 179 ExtensionAPIPermissionSet perms = info->GetAll();
179 int count = 0; 180 int count = 0;
180 for (ExtensionAPIPermissionSet::iterator i = perms.begin(); 181 for (ExtensionAPIPermissionSet::iterator i = perms.begin();
(...skipping 456 matching lines...) Expand 10 before | Expand all | Expand 10 after
637 // These are private. 638 // These are private.
638 skip.insert(ExtensionAPIPermission::kWebstorePrivate); 639 skip.insert(ExtensionAPIPermission::kWebstorePrivate);
639 skip.insert(ExtensionAPIPermission::kFileBrowserPrivate); 640 skip.insert(ExtensionAPIPermission::kFileBrowserPrivate);
640 skip.insert(ExtensionAPIPermission::kMediaPlayerPrivate); 641 skip.insert(ExtensionAPIPermission::kMediaPlayerPrivate);
641 skip.insert(ExtensionAPIPermission::kMetricsPrivate); 642 skip.insert(ExtensionAPIPermission::kMetricsPrivate);
642 skip.insert(ExtensionAPIPermission::kChromeAuthPrivate); 643 skip.insert(ExtensionAPIPermission::kChromeAuthPrivate);
643 skip.insert(ExtensionAPIPermission::kChromePrivate); 644 skip.insert(ExtensionAPIPermission::kChromePrivate);
644 skip.insert(ExtensionAPIPermission::kChromeosInfoPrivate); 645 skip.insert(ExtensionAPIPermission::kChromeosInfoPrivate);
645 skip.insert(ExtensionAPIPermission::kWebSocketProxyPrivate); 646 skip.insert(ExtensionAPIPermission::kWebSocketProxyPrivate);
646 skip.insert(ExtensionAPIPermission::kInputMethodPrivate); 647 skip.insert(ExtensionAPIPermission::kInputMethodPrivate);
648 skip.insert(ExtensionAPIPermission::kAccessibilityPrivate);
647 649
648 // Warned as part of host permissions. 650 // Warned as part of host permissions.
649 skip.insert(ExtensionAPIPermission::kDevtools); 651 skip.insert(ExtensionAPIPermission::kDevtools);
650 652
651 // Platform apps. TODO(miket): must we skip? 653 // Platform apps. TODO(miket): must we skip?
652 skip.insert(ExtensionAPIPermission::kSocket); 654 skip.insert(ExtensionAPIPermission::kSocket);
653 655
654 ExtensionPermissionsInfo* info = ExtensionPermissionsInfo::GetInstance(); 656 ExtensionPermissionsInfo* info = ExtensionPermissionsInfo::GetInstance();
655 ExtensionAPIPermissionSet permissions = info->GetAll(); 657 ExtensionAPIPermissionSet permissions = info->GetAll();
656 for (ExtensionAPIPermissionSet::const_iterator i = permissions.begin(); 658 for (ExtensionAPIPermissionSet::const_iterator i = permissions.begin();
(...skipping 440 matching lines...) Expand 10 before | Expand all | Expand 10 after
1097 AddPattern(&non_empty_extent, "http://www.google.com/*"); 1099 AddPattern(&non_empty_extent, "http://www.google.com/*");
1098 1100
1099 perm_set = new ExtensionPermissionSet( 1101 perm_set = new ExtensionPermissionSet(
1100 empty_apis, non_empty_extent, empty_extent); 1102 empty_apis, non_empty_extent, empty_extent);
1101 EXPECT_FALSE(perm_set->IsEmpty()); 1103 EXPECT_FALSE(perm_set->IsEmpty());
1102 1104
1103 perm_set = new ExtensionPermissionSet( 1105 perm_set = new ExtensionPermissionSet(
1104 empty_apis, empty_extent, non_empty_extent); 1106 empty_apis, empty_extent, non_empty_extent);
1105 EXPECT_FALSE(perm_set->IsEmpty()); 1107 EXPECT_FALSE(perm_set->IsEmpty());
1106 } 1108 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698