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

Unified Diff: chrome/browser/notifications/notification_interactive_uitest_support.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/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 4d8e157cc56ec9392be0284a5c91c414138e511f..e5168bd68276634a6ff6e1b12effaf98b4d8eaeb 100644
--- a/chrome/browser/notifications/notification_interactive_uitest_support.cc
+++ b/chrome/browser/notifications/notification_interactive_uitest_support.cc
@@ -117,14 +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)
feature_list_.InitAndDisableFeature(features::kNativeNotifications);
#endif // BUILDFLAG(ENABLE_NATIVE_NOTIFICATIONS)
+ InProcessBrowserTest::SetUp();
}
int NotificationsTest::GetNotificationCount() {

Powered by Google App Engine
This is Rietveld 408576698