Make unit_tests compile in Mac GN

BUILD.gn

 # Copyright (c) 2013 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.
 
 # This is the root build file for GN. GN will start processing by loading this
 # file, and recursively load all dependencies until all dependencies are either
 # resolved or known not to exist (which will cause the build to fail). So if
 # you add a new build file, there must be some path of dependencies from this
 # file to your new one or GN won't know about it.
 
@@ skipping 478 matching lines @@
   }
 
   if (is_mac) {
     deps += [
       "//breakpad:crash_inspector",
       "//breakpad:dump_syms",
       "//third_party/apple_sample_code",
       "//third_party/molokocacao",
     ]
 
-    # TODO(GYP): Remove these when the targets below work and these
-    # are pulled in automatically.
-    deps += [
-      "//cc/blink",
-      "//components/ui/zoom",
-      "//content",
-      "//content/test:test_support",
-      "//device/battery",
-      "//device/bluetooth",
-      "//device/nfc",
-      "//device/usb",
-      "//device/vibration",
-      "//media/blink",
-      "//pdf",
-      "//storage/browser",
-      "//third_party/brotli",
-      "//third_party/flac",
-      "//third_party/hunspell",
-      "//third_party/iccjpeg",
-      "//third_party/libphonenumber",
-      "//third_party/ots",
-      "//third_party/qcms",
-      "//third_party/smhasher:murmurhash3",
-      "//third_party/webrtc/system_wrappers",
-      "//ui/app_list:app_list_unittests",
-      "//ui/gfx:gfx_unittests",
-      "//ui/native_theme",
-      "//ui/snapshot",
-      "//ui/surface",
-    ]
-
-    # TODO(dpranke): These are as-yet untriaged but need at least the above.
+    # TODO(GYP): These don't work on Mac yet.
     deps -= [
       "//chrome",  # TODO(GYP)
       "//chrome/test:browser_tests",  # TODO(GYP)
       "//chrome/test:interactive_ui_tests",  # TODO(GYP)
       "//chrome/test:sync_integration_tests",  # TODO(GYP)
       "//chrome/test:unit_tests",  # TODO(GYP)

[Bons, 2015/10/21 13:54:09]

The CL description says make unit_tests compile, but this is still excluded. I
must be missing something?

[brettw, 2015/10/22 02:22:22]

No, I just need to remove this line. There are other tests in that BUILD file
that are referenced, so GN sees the unit_tests target even when it's excluded
here.

-      "//content/test:content_browsertests",  # TODO(GYP)
-      "//content/test:content_perftests",  # TODO(GYP)
       "//extensions:extensions_browsertests",  # TODO(GYP) Needs ContentMain
 
       # Mojo in GN contains some things which are never compiled in GYP on Mac,
       # so compilation fails on Mac. They need porting.
       "//mojo",  # TODO(GYP)
     ]
   }
 
   if (is_win) {
     deps += [
@@ skipping 274 matching lines @@
       if (target_cpu == "x86") {
         deps += [
           # "//content/shell:crash_service_win64", TODO(GYP): crbug.com/537009
         ]
       }
     } else {
       deps += [ "//breakpad:minidump_stackwalk($host_toolchain)" ]
     }
   }
 }