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

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

Issue 2910843002: [Cleanup] Move all browsertests to use ScopedFeatureList to modify features
Patch Set: comments addressed and updated 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/notifications/platform_notification_service_interactive_uitest.cc
diff --git a/chrome/browser/notifications/platform_notification_service_interactive_uitest.cc b/chrome/browser/notifications/platform_notification_service_interactive_uitest.cc
index 21a04dc228bc832783167fb48782ccd0aee8cdbc..13ee6c7a53a9a832db242a3149606c5060b6d720 100644
--- a/chrome/browser/notifications/platform_notification_service_interactive_uitest.cc
+++ b/chrome/browser/notifications/platform_notification_service_interactive_uitest.cc
@@ -144,13 +144,6 @@ void PlatformNotificationServiceBrowserTest::SetUpDefaultCommandLine(
// Needed for the Reply button tests
command_line->AppendSwitch(switches::kEnableExperimentalWebPlatformFeatures);
-
-#if BUILDFLAG(ENABLE_NATIVE_NOTIFICATIONS)
- // TODO(crbug.com/714679): Temporary change while tests are upgraded to deal
- // with native notifications.
- command_line->AppendSwitchASCII(switches::kDisableFeatures,
- features::kNativeNotifications.name);
-#endif // BUILDFLAG(ENABLE_NATIVE_NOTIFICATIONS)
}
void PlatformNotificationServiceBrowserTest::SetUp() {
@@ -159,6 +152,11 @@ void PlatformNotificationServiceBrowserTest::SetUp() {
new net::EmbeddedTestServer(net::EmbeddedTestServer::TYPE_HTTPS));
https_server_->ServeFilesFromSourceDirectory(server_root_);
ASSERT_TRUE(https_server_->Start());
+#if BUILDFLAG(ENABLE_NATIVE_NOTIFICATIONS)
+ // TODO(crbug.com/714679): Temporary change while tests are upgraded to deal
+ // with native notifications.
+ feature_list_.InitAndDisableFeature(features::kNativeNotifications);
+#endif // BUILDFLAG(ENABLE_NATIVE_NOTIFICATIONS)
InProcessBrowserTest::SetUp();
}

Powered by Google App Engine
This is Rietveld 408576698