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

Unified Diff: chrome/chrome_browser.gypi

Issue 1509923002: Implement native web notifications for mac (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years 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/chrome_browser.gypi
diff --git a/chrome/chrome_browser.gypi b/chrome/chrome_browser.gypi
index 2222ed04989465668f26ce78b81302190407937d..e2f196a8cc06bb9110daacd23c0ecedefaad5f94 100644
--- a/chrome/chrome_browser.gypi
+++ b/chrome/chrome_browser.gypi
@@ -2059,6 +2059,11 @@
'browser/push_messaging/push_messaging_notification_manager.cc',
'browser/push_messaging/push_messaging_notification_manager.h',
],
+ # Used on Mac when native notifications are enabled.
+ 'chrome_browser_notifications_mac_sources': [
Robert Sesek 2015/12/11 21:17:53 The GYP filename rules should filter this out, so
Miguel Garcia 2015/12/15 13:37:41 Done. Good idea
+ 'browser/notifications/notification_ui_manager_mac.h',
+ 'browser/notifications/notification_ui_manager_mac.mm',
+ ],
# Used on Android when notifications and java_ui are enabled.
'chrome_browser_notifications_android_java_ui_sources': [
'browser/notifications/notification_ui_manager_android.cc',
@@ -3639,6 +3644,11 @@
['notifications==1', {
'sources': [ '<@(chrome_browser_notifications_sources)' ],
'conditions': [
+ ['OS=="mac"', {
+ 'sources': [
+ '<@(chrome_browser_notifications_mac_sources)',
+ ],
+ }],
['android_java_ui==0', {
'sources': [
'<@(chrome_browser_notifications_non_android_sources)',

Powered by Google App Engine
This is Rietveld 408576698