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

Unified Diff: chrome/browser/browser_process_impl.cc

Issue 2033093003: [Notification] Make HTML5 Notification use ActionCenter on Windows 10, behind Flags. Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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/browser_process_impl.cc
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index 5c74de660fb31692169af2c391c3a7c1e5e7bba9..742e690e75d20504227035b4067b626e7fd31e51 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -575,7 +575,7 @@ NotificationUIManager* BrowserProcessImpl::notification_ui_manager() {
}
NotificationPlatformBridge* BrowserProcessImpl::notification_platform_bridge() {
-#if defined(OS_ANDROID) || defined(OS_MACOSX)
+#if defined(OS_WIN) || defined(OS_ANDROID) || defined(OS_MACOSX)
if (!created_notification_bridge_)
CreateNotificationPlatformBridge();
return notification_bridge_.get();
@@ -1062,11 +1062,13 @@ void BrowserProcessImpl::CreateIntranetRedirectDetector() {
}
void BrowserProcessImpl::CreateNotificationPlatformBridge() {
-#if (defined(OS_ANDROID) || defined(OS_MACOSX)) && defined(ENABLE_NOTIFICATIONS)
+#if defined(ENABLE_NOTIFICATIONS)
+#if defined(OS_WIN) || defined(OS_ANDROID) || defined(OS_MACOSX)
DCHECK(notification_bridge_.get() == NULL);
notification_bridge_.reset(NotificationPlatformBridge::Create());
created_notification_bridge_ = true;
#endif
+#endif
}
void BrowserProcessImpl::CreateNotificationUIManager() {

Powered by Google App Engine
This is Rietveld 408576698