[win/cros/lin] Add back the spellcheck menu.

chrome/chrome_browser.gypi

 # Copyright 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'variables': {
     # All .cc, .h, .m, and .mm files under browser except for:
     # * tests and mocks.
     # * code below browser/chromeos
     # * code below browser/extensions
@@ skipping 1018 matching lines @@
     # despite the change in direction in order to prevent iOS regressions while
     # componentization is in progress.
     # TODO(stuartmorgan): Remove this once enough has been componentized that
     # there's no value in building any of chrome/ for iOS.
     'chrome_browser_ios_sources': [
       'browser/browser_process.cc',
       'browser/browser_process.h',
       '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 indicating it by keyword
"desktop" in the group name.

+    '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',
       'browser/first_run/upgrade_util_linux.cc',
       'browser/first_run/upgrade_util_linux.h',
       'browser/fullscreen_aurax11.cc',
       'browser/icon_loader_auralinux.cc',
       'browser/password_manager/native_backend_kwallet_x.cc',
       'browser/password_manager/native_backend_kwallet_x.h',
       'browser/platform_util_linux.cc',
@@ skipping 2850 matching lines @@
             # x11 build
             ['use_x11==1', {
               'dependencies': [
                 '../build/linux/system.gyp:x11',
                 '../ui/events/devices/events_devices.gyp:events_devices',
                 '../ui/gfx/x/gfx_x11.gyp:gfx_x11',
               ],
             }],
           ],
         }],
+        ['OS=="linux" or OS=="win"', {

[please use gerrit instead, 2016/01/28 21:42:00]

Note that OS="linux" is also true for ChromeOS, but not true for Android,
although both of those are based on Linux.

[lazyboy, 2016/02/01 19:23:33]

Acknowledged.

+            'sources': [ '<@(chrome_browser_non_mac_desktop_sources)' ],
+        }],
         ['desktop_linux==1', {
           'sources': [ '<@(chrome_browser_linux_desktop_sources)' ],
         }],
         ['enable_plugin_installation==1', {
           'sources': [ '<@(chrome_browser_plugin_installation_sources)' ],
         }],
         ['enable_app_list==1', {
           'dependencies': [
             '../ui/app_list/app_list.gyp:app_list',
           ]
@@ skipping 272 matching lines @@
             '../components/components.gyp:omnibox_browser',
             '../components/components.gyp:rlz',
             '../components/components.gyp:search_engines',
             '../rlz/rlz.gyp:rlz_lib',
           ],
         },
       ],
     }],
   ],
 }