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

Unified Diff: chrome/browser/notifications/notification_permission_context_unittest.cc

Issue 2675483002: Replace PermissionType in chrome/ with ContentSettingsType (Closed)
Patch Set: rebase Created 3 years, 10 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/notifications/notification_permission_context_unittest.cc
diff --git a/chrome/browser/notifications/notification_permission_context_unittest.cc b/chrome/browser/notifications/notification_permission_context_unittest.cc
index d6239a6d97bc3f623b19ad05bffd09d2e6f7fbf2..1d0a3ac8be0362faf9cc05c3915fb1d35309a683 100644
--- a/chrome/browser/notifications/notification_permission_context_unittest.cc
+++ b/chrome/browser/notifications/notification_permission_context_unittest.cc
@@ -28,13 +28,13 @@
namespace {
void DoNothing(ContentSetting content_setting) {}
-void DoNothing2(blink::mojom::PermissionStatus status) {}
+void DoNothing2(blink::mojom::PermissionStatus content_setting) {}
class TestNotificationPermissionContext : public NotificationPermissionContext {
public:
explicit TestNotificationPermissionContext(Profile* profile)
: NotificationPermissionContext(profile,
- content::PermissionType::NOTIFICATIONS),
+ CONTENT_SETTINGS_TYPE_NOTIFICATIONS),
permission_set_count_(0),
last_permission_set_persisted_(false),
last_permission_set_setting_(CONTENT_SETTING_DEFAULT) {}
@@ -113,7 +113,7 @@ TEST_F(NotificationPermissionContextTest, IgnoresEmbedderOrigin) {
GURL different_origin("https://foobar.com");
NotificationPermissionContext context(profile(),
- content::PermissionType::NOTIFICATIONS);
+ CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
UpdateContentSetting(&context, requesting_origin, embedding_origin,
CONTENT_SETTING_ALLOW);
@@ -138,8 +138,8 @@ TEST_F(NotificationPermissionContextTest, PushTopLevelOriginOnly) {
GURL requesting_origin("https://example.com");
GURL embedding_origin("https://chrome.com");
- NotificationPermissionContext context(
- profile(), content::PermissionType::PUSH_MESSAGING);
+ NotificationPermissionContext context(profile(),
+ CONTENT_SETTINGS_TYPE_PUSH_MESSAGING);
UpdateContentSetting(&context, requesting_origin, embedding_origin,
CONTENT_SETTING_ALLOW);
@@ -166,7 +166,7 @@ TEST_F(NotificationPermissionContextTest, NoSecureOriginRequirement) {
GURL origin("http://example.com");
NotificationPermissionContext context(profile(),
- content::PermissionType::NOTIFICATIONS);
+ CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
EXPECT_EQ(CONTENT_SETTING_ASK,
context.GetPermissionStatus(origin, origin));
@@ -181,8 +181,8 @@ TEST_F(NotificationPermissionContextTest, PushSecureOriginRequirement) {
GURL origin("http://example.com");
GURL secure_origin("https://example.com");
- NotificationPermissionContext context(
- profile(), content::PermissionType::PUSH_MESSAGING);
+ NotificationPermissionContext context(profile(),
+ CONTENT_SETTINGS_TYPE_PUSH_MESSAGING);
EXPECT_EQ(CONTENT_SETTING_BLOCK, context.GetPermissionStatus(origin, origin));
UpdateContentSetting(&context, origin, origin, CONTENT_SETTING_ALLOW);
@@ -278,7 +278,7 @@ TEST_F(NotificationPermissionContextTest, TestCancelledIncognitoRequest) {
base::TestMockTimeTaskRunner* task_runner = SwitchToMockTime();
- content::PermissionManager* permission_manager =
+ PermissionManager* permission_manager =
PermissionManagerFactory::GetForProfile(
profile()->GetOffTheRecordProfile());
@@ -286,7 +286,7 @@ TEST_F(NotificationPermissionContextTest, TestCancelledIncognitoRequest) {
// https://crbug.com/586944 regresses, then as well as the EXPECT_EQs below
// failing, PermissionManager::OnPermissionsRequestResponseStatus will crash.
int request_id = permission_manager->RequestPermission(
- content::PermissionType::NOTIFICATIONS, web_contents()->GetMainFrame(),
+ CONTENT_SETTINGS_TYPE_NOTIFICATIONS, web_contents()->GetMainFrame(),
url.GetOrigin(), true /* user_gesture */, base::Bind(&DoNothing2));
permission_manager->CancelPermissionRequest(request_id);

Powered by Google App Engine
This is Rietveld 408576698