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

Unified Diff: extensions/common/permissions/permissions_data_unittest.cc

Issue 503873002: Remove implicit conversions from scoped_refptr to T* in extensions/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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: extensions/common/permissions/permissions_data_unittest.cc
diff --git a/extensions/common/permissions/permissions_data_unittest.cc b/extensions/common/permissions/permissions_data_unittest.cc
index 295f8d5183737f662a4124adb9f7923a104c9e17..33aef85e49a4e2c4e7c415297b42e18c7cc3c2d5 100644
--- a/extensions/common/permissions/permissions_data_unittest.cc
+++ b/extensions/common/permissions/permissions_data_unittest.cc
@@ -259,20 +259,19 @@ TEST(ExtensionPermissionsTest, IsRestrictedUrl) {
kAllHostsPermission,
Manifest::INTERNAL);
// Chrome urls should be blocked for normal extensions.
- CheckRestrictedUrls(extension, true);
+ CheckRestrictedUrls(extension.get(), true);
scoped_refptr<const Extension> component =
GetExtensionWithHostPermission("component",
kAllHostsPermission,
Manifest::COMPONENT);
// Chrome urls should be accessible by component extensions.
- CheckRestrictedUrls(component, false);
+ CheckRestrictedUrls(component.get(), false);
base::CommandLine::ForCurrentProcess()->AppendSwitch(
switches::kExtensionsOnChromeURLs);
// Enabling the switch should allow all extensions to access chrome urls.
- CheckRestrictedUrls(extension, false);
-
+ CheckRestrictedUrls(extension.get(), false);
}
TEST(ExtensionPermissionsTest, GetPermissionMessages_ManyAPIPermissions) {
« no previous file with comments | « extensions/common/permissions/permissions_data.cc ('k') | extensions/renderer/api/serial/data_receiver_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698