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

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

Issue 2910843002: [Cleanup] Move all browsertests to use ScopedFeatureList to modify features
Patch Set: Ilya comments addressed 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/notification_interactive_uitest_support.cc
diff --git a/chrome/browser/notifications/notification_interactive_uitest_support.cc b/chrome/browser/notifications/notification_interactive_uitest_support.cc
index dec10196b3fa7a7087650caeaac3dd894ada5447..e8b4ae7ee3479df9b38f545858f0110eb1b1ce15 100644
--- a/chrome/browser/notifications/notification_interactive_uitest_support.cc
+++ b/chrome/browser/notifications/notification_interactive_uitest_support.cc
@@ -117,15 +117,13 @@ bool MessageCenterChangeObserver::Wait() {
// -----------------------------------------------------------------------------
-void NotificationsTest::SetUpDefaultCommandLine(
- base::CommandLine* command_line) {
- InProcessBrowserTest::SetUpDefaultCommandLine(command_line);
+void NotificationsTest::SetUp() {
// Temporary change while the whole support class is changed to deal
// with native notifications. crbug.com/714679
#if BUILDFLAG(ENABLE_NATIVE_NOTIFICATIONS)
- command_line->AppendSwitchASCII(switches::kDisableFeatures,
- features::kNativeNotifications.name);
+ feature_list_.InitAndDisableFeature(features::kNativeNotifications);
#endif // BUILDFLAG(ENABLE_NATIVE_NOTIFICATIONS)
+ InProcessBrowserTest::SetUp();
}
int NotificationsTest::GetNotificationCount() {

Powered by Google App Engine
This is Rietveld 408576698