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

Unified Diff: chrome/chrome_browser.gypi

Issue 1647723002: [win/cros/lin] Add back the spellcheck menu. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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/chrome_browser.gypi
diff --git a/chrome/chrome_browser.gypi b/chrome/chrome_browser.gypi
index a4cf99ea39255fc21721cfdc9823c70b02a11328..1a31e25fc3570cbd70ff3b43493cf52d642e847d 100644
--- a/chrome/chrome_browser.gypi
+++ b/chrome/chrome_browser.gypi
@@ -1036,6 +1036,11 @@
'browser/browser_process_platform_part_base.cc',
'browser/browser_process_platform_part_base.h',
],
+ # Desktop Linux, ChromeOS, and Windows.
lazyboy 2016/01/28 20:36:25 Android and ios?
please use gerrit instead 2016/01/28 21:42:00 Android and iOS do not have this submenu. I am ind
+ 'chrome_browser_non_mac_desktop_sources': [
+ 'browser/renderer_context_menu/spellchecker_submenu_observer.h',
+ 'browser/renderer_context_menu/spellchecker_submenu_observer.cc',
+ ],
# Desktop linux, doesn't count ChromeOS.
'chrome_browser_linux_desktop_sources': [
'browser/first_run/upgrade_util.cc',
@@ -3906,6 +3911,9 @@
}],
],
}],
+ ['OS=="linux" or OS=="win"', {
please use gerrit instead 2016/01/28 21:42:00 Note that OS="linux" is also true for ChromeOS, bu
lazyboy 2016/02/01 19:23:33 Acknowledged.
+ 'sources': [ '<@(chrome_browser_non_mac_desktop_sources)' ],
+ }],
['desktop_linux==1', {
'sources': [ '<@(chrome_browser_linux_desktop_sources)' ],
}],

Powered by Google App Engine
This is Rietveld 408576698