Fix some TODO(GYP) in the GN build.

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 163 matching lines @@
       "//ppapi/examples/scaling",
       "//ppapi/examples/scripting",
       "//ppapi/examples/stub",
       "//ppapi/examples/threading",
       "//ppapi/examples/url_loader",
       "//ppapi/examples/video_capture",
       "//ppapi/examples/video_decode",
       "//ppapi/examples/video_effects",
       "//ppapi/examples/video_encode",
       "//printing:printing_unittests",
+      "//sync/tools:sync_client",

[brettw, 2016/05/23 22:06:25]

Moved from below (compiled on more target types now)

+      "//sync/tools:sync_listen_notifications",
       "//third_party/cacheinvalidation:cacheinvalidation_unittests",
       "//third_party/codesighs",
       "//third_party/pdfium/samples:pdfium_test",
       "//tools/battor_agent",
       "//tools/battor_agent:battor_agent_unittests",
       "//tools/gn",
       "//tools/gn:gn_unittests",
       "//tools/perf/clear_system_cache",
       "//ui/accessibility:accessibility_unittests",
     ]
@@ skipping 328 matching lines @@
 
     if (!is_android) {
       deps += [
         "//build/sanitizers:copy_llvm_symbolizer",
         "//chrome/test:chrome_app_unittests",
         "//gpu/khronos_glcts_support:khronos_glcts_test",
         "//media/cast:cast_benchmarks",
         "//media/cast:tap_proxy",
         "//skia:filter_fuzz_stub",
         "//skia:image_operations_bench",
-        "//sync/tools:sync_client",
-        "//sync/tools:sync_listen_notifications",
         "//third_party/sqlite:sqlite_shell",
         "//ui/keyboard:keyboard_unittests",
         "//ui/message_center:message_center_unittests",
         "//ui/snapshot:snapshot_unittests",
         "//ui/views/examples:views_examples_with_content_exe",
       ]
 
       if (!is_debug && !is_component_build) {
         deps += [ "//chrome/tools/service_discovery_sniffer" ]
       }
@@ skipping 239 matching lines @@
       "//media/mojo:tests",
       "//media/mojo/services",
       "//mojo:tests",
     ]
   }
 }
 
 group("gn_visibility") {
   deps = [
     "//build/config/sanitizers:options_sources",
-
     # "//third_party/pdfium:pdfium_embeddertests",  # TODO(GYP): visibility?
     # "//third_party/pdfium:pdfium_unittests",  # TODO(GYP): visibility?
-
-    "//ui/resources:repack_ui_test_mac_locale_pack",

[brettw, 2016/05/23 22:06:25]

This is referenced organically now so this reference is not needed.

   ]
 
   if (!is_ios) {
     deps += [
       "//v8:postmortem-metadata",
       "//v8:v8_snapshot",
     ]
   }
 }
 
@@ skipping 140 matching lines @@
   assert(target_name != "")  # Mark as used.
   sources = invoker.actual_sources
   assert(
       sources == invoker.actual_sources,
       "Do not use a platform name in your output directory (found \"$root_build_dir\"). http://crbug.com/548283")
 }
 
 assert_valid_out_dir("_unused") {
   actual_sources = [ "$root_build_dir/foo" ]
 }