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

Unified Diff: chrome/browser/extensions/api/permissions/permissions_api_helpers_unittest.cc

Issue 228343002: Cleanup webRequest/Internal API and permission registration. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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/browser/extensions/api/permissions/permissions_api_helpers_unittest.cc
diff --git a/chrome/browser/extensions/api/permissions/permissions_api_helpers_unittest.cc b/chrome/browser/extensions/api/permissions/permissions_api_helpers_unittest.cc
index 3ab2d411c2330f9a26a3a684f282f686e0197dfd..f932eeac72cea02e211e4101b20c3bcfc747c012 100644
--- a/chrome/browser/extensions/api/permissions/permissions_api_helpers_unittest.cc
+++ b/chrome/browser/extensions/api/permissions/permissions_api_helpers_unittest.cc
@@ -30,8 +30,8 @@ static void AddPattern(URLPatternSet* extent, const std::string& pattern) {
TEST(ExtensionPermissionsAPIHelpers, Pack) {
APIPermissionSet apis;
apis.insert(APIPermission::kTab);
- apis.insert(APIPermission::kWebRequest);
- // Note: kWebRequest implies also kWebRequestInternal.
+ apis.insert(APIPermission::kFileBrowserHandler);
+ // Note: kFileBrowserHandler implies kFileBrowserHandlerInternal.
URLPatternSet hosts;
AddPattern(&hosts, "http://a.com/*");
AddPattern(&hosts, "http://b.com/*");
@@ -50,7 +50,8 @@ TEST(ExtensionPermissionsAPIHelpers, Pack) {
EXPECT_EQ(3u, api_list->GetSize());
EXPECT_EQ(2u, origin_list->GetSize());
- std::string expected_apis[] = { "tabs", "webRequest" };
+ std::string expected_apis[] = {"tabs", "fileBrowserHandler",
+ "fileBrowserHandlerInternal"};
for (size_t i = 0; i < ARRAYSIZE_UNSAFE(expected_apis); ++i) {
scoped_ptr<base::Value> value(new base::StringValue(expected_apis[i]));
EXPECT_NE(api_list->end(), api_list->Find(*value));
« no previous file with comments | « no previous file | chrome/common/extensions/api/_api_features.json » ('j') | chrome/common/extensions/api/web_request_internal.json » ('J')

Powered by Google App Engine
This is Rietveld 408576698