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

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

Issue 8071025: Move contentSettings extension API out of experimental (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix Created 9 years, 2 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 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 592 matching lines...) Expand 10 before | Expand all | Expand 10 after
603 ExtensionAPIPermissionSet skip; 603 ExtensionAPIPermissionSet skip;
604 604
605 skip.insert(ExtensionAPIPermission::kDefault); 605 skip.insert(ExtensionAPIPermission::kDefault);
606 606
607 // These are considered "nuisance" or "trivial" permissions that don't need 607 // These are considered "nuisance" or "trivial" permissions that don't need
608 // a prompt. 608 // a prompt.
609 skip.insert(ExtensionAPIPermission::kContextMenus); 609 skip.insert(ExtensionAPIPermission::kContextMenus);
610 skip.insert(ExtensionAPIPermission::kIdle); 610 skip.insert(ExtensionAPIPermission::kIdle);
611 skip.insert(ExtensionAPIPermission::kNotification); 611 skip.insert(ExtensionAPIPermission::kNotification);
612 skip.insert(ExtensionAPIPermission::kUnlimitedStorage); 612 skip.insert(ExtensionAPIPermission::kUnlimitedStorage);
613 skip.insert(ExtensionAPIPermission::kContentSettings);
614 skip.insert(ExtensionAPIPermission::kTts); 613 skip.insert(ExtensionAPIPermission::kTts);
615 614
616 // TODO(erikkay) add a string for this permission. 615 // TODO(erikkay) add a string for this permission.
617 skip.insert(ExtensionAPIPermission::kBackground); 616 skip.insert(ExtensionAPIPermission::kBackground);
618 617
619 skip.insert(ExtensionAPIPermission::kClipboardWrite); 618 skip.insert(ExtensionAPIPermission::kClipboardWrite);
620 619
621 // The cookie permission does nothing unless you have associated host 620 // The cookie permission does nothing unless you have associated host
622 // permissions. 621 // permissions.
623 skip.insert(ExtensionAPIPermission::kCookie); 622 skip.insert(ExtensionAPIPermission::kCookie);
(...skipping 468 matching lines...) Expand 10 before | Expand all | Expand 10 after
1092 AddPattern(&non_empty_extent, "http://www.google.com/*"); 1091 AddPattern(&non_empty_extent, "http://www.google.com/*");
1093 1092
1094 perm_set = new ExtensionPermissionSet( 1093 perm_set = new ExtensionPermissionSet(
1095 empty_apis, non_empty_extent, empty_extent); 1094 empty_apis, non_empty_extent, empty_extent);
1096 EXPECT_FALSE(perm_set->IsEmpty()); 1095 EXPECT_FALSE(perm_set->IsEmpty());
1097 1096
1098 perm_set = new ExtensionPermissionSet( 1097 perm_set = new ExtensionPermissionSet(
1099 empty_apis, empty_extent, non_empty_extent); 1098 empty_apis, empty_extent, non_empty_extent);
1100 EXPECT_FALSE(perm_set->IsEmpty()); 1099 EXPECT_FALSE(perm_set->IsEmpty());
1101 } 1100 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698