Cleanup inconsistencies between GN and GYP in //remoting

remoting/test/BUILD.gn

Index: remoting/test/BUILD.gn
diff --git a/remoting/test/BUILD.gn b/remoting/test/BUILD.gn
new file mode 100644
index 0000000000000000000000000000000000000000..4b4e75110954a1415b5b5bc7ee4ab8778e08deb7
--- /dev/null
+++ b/remoting/test/BUILD.gn
@@ -0,0 +1,121 @@
+# 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.
+
+source_set("test_support") {
+  sources = [
+    "access_token_fetcher.cc",
+    "access_token_fetcher.h",
+    "app_remoting_connected_client_fixture.cc",
+    "app_remoting_connected_client_fixture.h",
+    "app_remoting_connection_helper.cc",
+    "app_remoting_connection_helper.h",
+    "app_remoting_latency_test_fixture.cc",
+    "app_remoting_latency_test_fixture.h",
+    "app_remoting_report_issue_request.cc",
+    "app_remoting_report_issue_request.h",
+    "app_remoting_service_urls.cc",
+    "app_remoting_service_urls.h",
+    "app_remoting_test_driver_environment.cc",
+    "app_remoting_test_driver_environment.h",
+    "app_remoting_test_driver_environment_app_details.cc",
+    "fake_access_token_fetcher.cc",
+    "fake_access_token_fetcher.h",
+    "fake_app_remoting_report_issue_request.cc",
+    "fake_app_remoting_report_issue_request.h",
+    "fake_network_dispatcher.cc",
+    "fake_network_dispatcher.h",
+    "fake_network_manager.cc",
+    "fake_network_manager.h",
+    "fake_port_allocator.cc",
+    "fake_port_allocator.h",
+    "fake_remote_host_info_fetcher.cc",
+    "fake_remote_host_info_fetcher.h",
+    "fake_socket_factory.cc",
+    "fake_socket_factory.h",
+    "host_info.cc",
+    "host_info.h",
+    "host_list_fetcher.cc",
+    "host_list_fetcher.h",
+    "leaky_bucket.cc",
+    "leaky_bucket.h",
+    "mock_access_token_fetcher.cc",
+    "mock_access_token_fetcher.h",
+    "refresh_token_store.cc",
+    "refresh_token_store.h",
+    "remote_application_details.h",
+    "remote_connection_observer.h",
+    "remote_host_info.cc",
+    "remote_host_info.h",
+    "remote_host_info_fetcher.cc",
+    "remote_host_info_fetcher.h",
+    "test_chromoting_client.cc",
+    "test_chromoting_client.h",
+    "test_video_renderer.cc",
+    "test_video_renderer.h",
+  ]
+
+  public_deps = [
+    "//base",
+    "//net",
+    "//remoting/base",
+    "//remoting/protocol",
+    "//remoting/codec",
+    "//remoting/client",
+    "//remoting/signaling",
+  ]
+
+  deps = [
+    "//google_apis",
+    "//testing/gmock",
+    "//testing/gtest",
+    "//third_party/libjingle",
+  ]
+}
+
+source_set("unit_tests") {
+  testonly = true
+
+  sources = [
+    "access_token_fetcher_unittest.cc",
+    "app_remoting_report_issue_request_unittest.cc",
+    "app_remoting_test_driver_environment_unittest.cc",
+    "host_list_fetcher_unittest.cc",
+    "remote_host_info_fetcher_unittest.cc",
+    "test_chromoting_client_unittest.cc",
+    "test_video_renderer_unittest.cc",
+  ]
+
+  deps = [
+    ":test_support",
+    "//base",
+    "//net:test_support",
+    "//testing/gmock",
+    "//testing/gtest",
+    "//third_party/libyuv",
+    "//third_party/webrtc/modules/desktop_capture",
+  ]
+}
+
+executable("chromoting_test_driver") {
+  sources = [
+    "chromoting_test_driver.cc",
+  ]

[joedow, 2015/07/16 02:45:14]

nit: the newlines between blocks seems inconsistent (in all of the files),
should there be breaks in between blocks (or is there a formatter that will
handle it automatically)?

[Sergey Ulanov, 2015/07/16 17:39:12]

There is a formatter for gn, but it doesn't insert or remove these empty lines

+  deps = [
+    ":test_support",
+    "//base/test:test_support",
+    "//testing/gtest",
+  ]
+}
+
+# An external version of the test driver tool which includes minimal tests
+executable("ar_sample_test_driver") {
+  sources = [
+    "app_remoting_test_driver.cc",
+  ]
+  deps = [
+    ":test_support",
+    "//base/test:test_support",
+    "//testing/gtest",
+  ]
+}