Enable MacOSX native notifications by default

chrome/browser/notifications/notification_interactive_uitest_support.cc

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 73680f9a35c053d7182f219bc01404b4395145c4..aac57fe28f59fad5c977bfdd68102a012e2c590f 100644
--- a/chrome/browser/notifications/notification_interactive_uitest_support.cc
+++ b/chrome/browser/notifications/notification_interactive_uitest_support.cc
@@ -116,6 +116,16 @@ bool MessageCenterChangeObserver::Wait() {
 
 // -----------------------------------------------------------------------------
 
+void NotificationsTest::SetUpDefaultCommandLine(

[Peter Beverloo, 2017/04/24 16:11:16]

I think it's more common to override SetUpCommandLine()

[Miguel Garcia, 2017/04/24 17:26:04]

Acknowledged.

+    base::CommandLine* command_line) {
+  InProcessBrowserTest::SetUpDefaultCommandLine(command_line);
+// Temporary change while the whole support class is changed to deal
+// with native notifications.

[Peter Beverloo, 2017/04/24 16:11:16]

nit: refer to a bug

[Miguel Garcia, 2017/04/24 17:26:04]

Done.

+#if defined(OS_MACOSX)
+  command_line->AppendSwitchASCII("disable-features", "NativeNotifications");

[Peter Beverloo, 2017/04/24 16:11:16]

#include "chrome/common/chrome_features.h"
#include "content/public/common/content_switches.h"

"disable-features" -> content::kDisableFeatures
"NativeNotifications" -> features::kNativeNotifications.name

[Miguel Garcia, 2017/04/24 17:26:04]

Done.

+#endif
+}
+
 int NotificationsTest::GetNotificationCount() {
   return message_center::MessageCenter::Get()->NotificationCount();
 }