MacViews: Views accelerators table should match the Cocoa one.

chrome/browser/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/chrome_build.gni")
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//build/split_static_library.gni")
 import("//chrome/common/features.gni")
@@ skipping 3749 matching lines @@
   if (enable_wayland_server) {
     deps += [
       "//components/exo",
       "//components/exo/wayland",
     ]
     sources += [
       "chrome_browser_main_extra_parts_exo.cc",
       "chrome_browser_main_extra_parts_exo.h",
     ]
   }
+
+  if (is_mac) {

[tapted, 2016/10/31 10:16:57]

this should be merged with the if (is_mac) on line 3394

[themblsha, 2016/10/31 16:56:29]

Thanks. I'm getting lost in these huge GN files.

+    if (mac_views_browser) {
+      sources += [ "global_keyboard_shortcuts_views_mac.mm" ]
+    } else {
+      sources += [ "global_keyboard_shortcuts_cocoa_mac.mm" ]
+    }
+  }
 }
 
 if (android_java_ui) {
   generate_jni("jni_headers") {
     sources = [
       "../android/java/src/org/chromium/chrome/browser/AfterStartupTaskUtils.java",
       "../android/java/src/org/chromium/chrome/browser/ApplicationLifetime.java",
       "../android/java/src/org/chromium/chrome/browser/BackgroundSyncLauncher.java",
       "../android/java/src/org/chromium/chrome/browser/BluetoothChooserDialog.java",
       "../android/java/src/org/chromium/chrome/browser/ChromeApplication.java",
@@ skipping 579 matching lines @@
   # linking all of the test support.
   static_library("pepper_cdm_test_constants") {
     testonly = true
     visibility = [ "//chrome/*" ]
     sources = [
       "media/pepper_cdm_test_constants.cc",
       "media/pepper_cdm_test_constants.h",
     ]
   }
 }