Cleanups in remoting gn files.

remoting/test/BUILD.gn

 # Copyright 2015 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("//remoting/remoting_enable.gni")
+import("//remoting/remoting_options.gni")
 
 static_library("test_support") {
   testonly = true
 
   sources = [
     "access_token_fetcher.cc",
     "access_token_fetcher.h",
     "app_remoting_report_issue_request.cc",
     "app_remoting_report_issue_request.h",
     "app_remoting_service_urls.cc",
@@ skipping 50 matching lines @@
     "test_video_renderer.h",
     "video_frame_writer.cc",
     "video_frame_writer.h",
   ]
 
   public_deps = [
     "//base",
     "//net",
     "//remoting/base",
     "//remoting/client",
-    "//remoting/codec",
+    "//remoting/codec:decoder",
+    "//remoting/codec:encoder",

[nicholss, 2017/01/20 23:05:22]

If you leave the default target for //remoting/codec you don't need to have
both. Or move the decoder and encoder target to public deps in the :codec target
and it will trickle up as you expect.

[Sergey Ulanov, 2017/01/20 23:41:41]

The goal is to force every target that needs encoders to depend on in explicitly
as it pulls media and ffmpeg deps, that are not needed on the client side.

     "//remoting/protocol",
     "//remoting/signaling",
     "//third_party/webrtc/modules/desktop_capture",
     "//ui/gfx",
   ]
 
   deps = [
     "//google_apis",
     "//skia",
     "//testing/gmock",
@@ skipping 141 matching lines @@
     ":ar_test_driver_common",
     ":test_support",
     "//base",
     "//net:test_support",
     "//testing/gmock",
     "//testing/gtest",
     "//third_party/libyuv",
     "//third_party/webrtc/modules/desktop_capture",
   ]
 }