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

Unified Diff: chrome/browser/browser_process_impl.cc

Issue 2794103002: Add initial support for native Linux desktop notifications (Closed)
Patch Set: Remove unused import Created 3 years, 8 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
« no previous file with comments | « chrome/browser/about_flags.cc ('k') | chrome/browser/notifications/notification_display_service_factory.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browser_process_impl.cc
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index 34edc3caa532928701879a7588c463a7aa92ac8d..39dc70edf71dbc9bbb7802e5129715c43cb1c344 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -633,7 +633,7 @@ NotificationUIManager* BrowserProcessImpl::notification_ui_manager() {
}
NotificationPlatformBridge* BrowserProcessImpl::notification_platform_bridge() {
-#if defined(OS_ANDROID) || defined(OS_MACOSX)
+#if BUILDFLAG(ENABLE_NATIVE_NOTIFICATIONS)
if (!created_notification_bridge_)
CreateNotificationPlatformBridge();
return notification_bridge_.get();
@@ -1162,7 +1162,7 @@ void BrowserProcessImpl::CreateIntranetRedirectDetector() {
}
void BrowserProcessImpl::CreateNotificationPlatformBridge() {
-#if (defined(OS_ANDROID) || defined(OS_MACOSX))
+#if BUILDFLAG(ENABLE_NATIVE_NOTIFICATIONS)
DCHECK(!notification_bridge_);
notification_bridge_.reset(NotificationPlatformBridge::Create());
created_notification_bridge_ = true;
« no previous file with comments | « chrome/browser/about_flags.cc ('k') | chrome/browser/notifications/notification_display_service_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698