[MacViews] Get unit_tests linking.

chrome/browser/ui/BUILD.gn

Index: chrome/browser/ui/BUILD.gn
diff --git a/chrome/browser/ui/BUILD.gn b/chrome/browser/ui/BUILD.gn
index 91825ea9297fd10277fad7138a38ea35ee4888aa..067325e3fa5be704ba600e96a96f55029bd3dc6c 100644
--- a/chrome/browser/ui/BUILD.gn
+++ b/chrome/browser/ui/BUILD.gn
@@ -555,8 +555,8 @@ source_set("test_support") {
 
   sources = [
     "browser.h",
-    "cocoa/extensions/browser_action_test_util_mac.mm",
-    "cocoa/find_bar/find_bar_host_unittest_util_cocoa.mm",
+    "cocoa/cocoa_test_helper.h",
+    "cocoa/cocoa_test_helper.mm",
     "cocoa/run_loop_testing.h",
     "cocoa/run_loop_testing.mm",
     "exclusive_access/fullscreen_controller_state_test.cc",
@@ -601,6 +601,13 @@ source_set("test_support") {
     ]
   }
 
+  if (is_mac && !mac_views_browser) {

[tapted, 2015/12/17 01:49:19]

can this just be an `else` clause of `if (toolkit_views && (!is_mac ||
mac_views_browser))` on line 515?  (i.e. and still rely on the
platform-stripping for non-mac). This feels appropriate since we'd never want to
include both file groups on any platform.

[jackhou1, 2015/12/17 04:03:50]

Done.

+    sources += [
+      "cocoa/extensions/browser_action_test_util_mac.mm",
+      "cocoa/find_bar/find_bar_host_unittest_util_cocoa.mm",
+    ]
+  }
+
   public_deps = [
     ":ui",
   ]