[Chrome OS] Move initialization of MessageCenter singleton into Ash.
also,
- use native notifications any time Mash is enabled
- get rid of ScopedKeepAlive in ProfileNotification for Chrome OS
and the call to clear ScopedKeepAlives in HandleAppExitingForPlatform.
This should have no effect on a normal run of Chrome, where the
browser process is kept alive regardless of how many browser windows
are left, but it's necessary to do this for browser tests to
terminate cleanly without jumping through the hoops of manually
destroying existing ProfileNotification objects.
Change-Id: I510ec4e774965159ee03209791f6bd1c43c3c042
Reviewed-on: https://chromium-review.googlesource.com/940186
Reviewed-by: Scott Violet <[email protected]>
Reviewed-by: Steven Bennetts <[email protected]>
Reviewed-by: James Cook <[email protected]>
Commit-Queue: Evan Stade <[email protected]>
Cr-Commit-Position: refs/heads/master@{#542884}diff --git a/chrome/browser/DEPS b/chrome/browser/DEPS
index 40b6df11..c450a8a1 100644
--- a/chrome/browser/DEPS
+++ b/chrome/browser/DEPS
@@ -89,6 +89,10 @@
"+ash/public",
"+ash/components/quick_launch/public/mojom",
+ # The message center may not exist, or may run out of process (in mash).
+ "-ui/message_center",
+ "+ui/message_center/public",
+
# chrome only needs switches from cc. All usage of the compositor is from
# content. Definitely don't include generic stuff from cc/base here, if this
# is needed these files need to move to base/
@@ -206,5 +210,10 @@
],
"display_info_provider_chromeos_unittest.cc": [
"+ui/events/devices/touch_device_transform.h"
- ]
+ ],
+ # TODO(estade): the message center is initialized and shut down here, but it
+ # should be the responsibility of code in chrome/browser/notifications/
+ "browser_process_impl.cc": [
+ "+ui/message_center/message_center.h",
+ ],
}
