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

Unified Diff: chrome/browser/extensions/api/web_request/web_request_permissions_unittest.cc

Issue 2930983002: Revert of chrome.webRequest support for ExtensionSettings (Closed)
Patch Set: Created 3 years, 6 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/web_request/web_request_permissions_unittest.cc
diff --git a/chrome/browser/extensions/api/web_request/web_request_permissions_unittest.cc b/chrome/browser/extensions/api/web_request/web_request_permissions_unittest.cc
index 0303a492e24f60c8db9d8afce2dd5dd28cc8be42..ed19ffdd02c48e1858303e80e33dbd5e3e612f06 100644
--- a/chrome/browser/extensions/api/web_request/web_request_permissions_unittest.cc
+++ b/chrome/browser/extensions/api/web_request/web_request_permissions_unittest.cc
@@ -181,35 +181,32 @@
context.CreateRequest(GURL("http://example.com"), net::DEFAULT_PRIORITY,
NULL, TRAFFIC_ANNOTATION_FOR_TESTS));
- EXPECT_EQ(
- PermissionsData::ACCESS_ALLOWED,
- WebRequestPermissions::CanExtensionAccessURL(
- extension_info_map_.get(), permissionless_extension_->id(),
- request->url(),
- -1, // No tab id.
- false, // crosses_incognito
- WebRequestPermissions::DO_NOT_CHECK_HOST, request->initiator()));
- EXPECT_EQ(PermissionsData::ACCESS_DENIED,
+ EXPECT_EQ(PermissionsData::ACCESS_ALLOWED,
WebRequestPermissions::CanExtensionAccessURL(
extension_info_map_.get(), permissionless_extension_->id(),
request->url(),
- -1, // No tab id.
- false, // crosses_incognito
- WebRequestPermissions::REQUIRE_HOST_PERMISSION,
- request->initiator()));
+ -1, // No tab id.
+ false, // crosses_incognito
+ WebRequestPermissions::DO_NOT_CHECK_HOST));
+ EXPECT_EQ(PermissionsData::ACCESS_DENIED,
+ WebRequestPermissions::CanExtensionAccessURL(
+ extension_info_map_.get(), permissionless_extension_->id(),
+ request->url(),
+ -1, // No tab id.
+ false, // crosses_incognito
+ WebRequestPermissions::REQUIRE_HOST_PERMISSION));
EXPECT_EQ(PermissionsData::ACCESS_ALLOWED,
WebRequestPermissions::CanExtensionAccessURL(
extension_info_map_.get(), com_extension_->id(), request->url(),
- -1, // No tab id.
- false, // crosses_incognito
- WebRequestPermissions::REQUIRE_HOST_PERMISSION,
- request->initiator()));
+ -1, // No tab id.
+ false, // crosses_incognito
+ WebRequestPermissions::REQUIRE_HOST_PERMISSION));
EXPECT_EQ(PermissionsData::ACCESS_DENIED,
WebRequestPermissions::CanExtensionAccessURL(
extension_info_map_.get(), com_extension_->id(), request->url(),
- -1, // No tab id.
- false, // crosses_incognito
- WebRequestPermissions::REQUIRE_ALL_URLS, request->initiator()));
+ -1, // No tab id.
+ false, // crosses_incognito
+ WebRequestPermissions::REQUIRE_ALL_URLS));
// Public Sessions tests.
#if defined(OS_CHROMEOS)
@@ -221,10 +218,9 @@
WebRequestPermissions::CanExtensionAccessURL(
extension_info_map_.get(), com_policy_extension_->id(),
org_request->url(),
- -1, // No tab id.
- false, // crosses_incognito
- WebRequestPermissions::REQUIRE_HOST_PERMISSION,
- org_request->initiator()));
+ -1, // No tab id.
+ false, // crosses_incognito
+ WebRequestPermissions::REQUIRE_HOST_PERMISSION));
chromeos::ScopedTestPublicSessionLoginState login_state;
@@ -234,19 +230,17 @@
WebRequestPermissions::CanExtensionAccessURL(
extension_info_map_.get(), com_policy_extension_->id(),
org_request->url(),
- -1, // No tab id.
- false, // crosses_incognito
- WebRequestPermissions::REQUIRE_HOST_PERMISSION,
- org_request->initiator()));
-
- EXPECT_EQ(
- PermissionsData::ACCESS_ALLOWED,
- WebRequestPermissions::CanExtensionAccessURL(
- extension_info_map_.get(), com_policy_extension_->id(),
- org_request->url(),
- -1, // No tab id.
- false, // crosses_incognito
- WebRequestPermissions::REQUIRE_ALL_URLS, org_request->initiator()));
+ -1, // No tab id.
+ false, // crosses_incognito
+ WebRequestPermissions::REQUIRE_HOST_PERMISSION));
+
+ EXPECT_EQ(PermissionsData::ACCESS_ALLOWED,
+ WebRequestPermissions::CanExtensionAccessURL(
+ extension_info_map_.get(), com_policy_extension_->id(),
+ org_request->url(),
+ -1, // No tab id.
+ false, // crosses_incognito
+ WebRequestPermissions::REQUIRE_ALL_URLS));
// Make sure that chrome:// URLs cannot be accessed.
std::unique_ptr<net::URLRequest> chrome_request(
@@ -257,9 +251,8 @@
WebRequestPermissions::CanExtensionAccessURL(
extension_info_map_.get(), com_policy_extension_->id(),
chrome_request->url(),
- -1, // No tab id.
- false, // crosses_incognito
- WebRequestPermissions::REQUIRE_HOST_PERMISSION,
- chrome_request->initiator()));
+ -1, // No tab id.
+ false, // crosses_incognito
+ WebRequestPermissions::REQUIRE_HOST_PERMISSION));
#endif
}

Powered by Google App Engine
This is Rietveld 408576698