Commit ecf455be authored by Evan Stade's avatar Evan Stade Committed by Commit Bot

Reland "Turn on native notifications by default for Chrome OS."

This is a reland of 8ce1863f

MSan fix was landed as 522e7d29

Original change's description:
> Turn on native notifications by default for Chrome OS.
>
> They're already on by default for Mash. This makes Chrome use the mojo
> ("native") interface for notifications even in non-Mash.
>
> Bug: 578868
> Change-Id: If1ee85e2136dc0480ccee9a57835b0ffb9b47b8b
> Reviewed-on: https://chromium-review.googlesource.com/963518
> Commit-Queue: Evan Stade <estade@chromium.org>
> Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#544395}

Bug: 578868
Change-Id: Idac45274aaea709ef910d4bcb20b14f9a05a952c
Reviewed-on: https://chromium-review.googlesource.com/974201Reviewed-by: default avatarScott Violet <sky@chromium.org>
Commit-Queue: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545113}
parent 338fdc53
......@@ -406,10 +406,9 @@ const base::Feature kMultiDeviceApi{"MultiDeviceApi",
#endif
// Enables the use of native notification centers instead of using the Message
// Center for displaying the toasts.
// Center for displaying the toasts. Note that OS_LINUX includes Chrome OS.
#if BUILDFLAG(ENABLE_NATIVE_NOTIFICATIONS)
#if defined(OS_MACOSX) || defined(OS_ANDROID) || \
(defined(OS_LINUX) && !defined(OS_CHROMEOS))
#if defined(OS_MACOSX) || defined(OS_ANDROID) || defined(OS_LINUX)
const base::Feature kNativeNotifications{"NativeNotifications",
base::FEATURE_ENABLED_BY_DEFAULT};
#else
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment