Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(218)

Side by Side Diff: remoting/test/BUILD.gn

Issue 1238343002: Added ConnectionTimeObserver to calculate the times to authenticate and connect. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added unittest, display connection stats ability, and error check for transition delay. Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 source_set("test_support") { 5 source_set("test_support") {
6 testonly = true 6 testonly = true
7 7
8 sources = [ 8 sources = [
9 "access_token_fetcher.cc", 9 "access_token_fetcher.cc",
10 "access_token_fetcher.h", 10 "access_token_fetcher.h",
11 "app_remoting_report_issue_request.cc", 11 "app_remoting_report_issue_request.cc",
12 "app_remoting_report_issue_request.h", 12 "app_remoting_report_issue_request.h",
13 "app_remoting_service_urls.cc", 13 "app_remoting_service_urls.cc",
14 "app_remoting_service_urls.h", 14 "app_remoting_service_urls.h",
15 "connection_setup_info.cc", 15 "connection_setup_info.cc",
16 "connection_setup_info.h", 16 "connection_setup_info.h",
17 "connection_time_observer.cc",
18 "connection_time_observer.h",
17 "fake_access_token_fetcher.cc", 19 "fake_access_token_fetcher.cc",
18 "fake_access_token_fetcher.h", 20 "fake_access_token_fetcher.h",
19 "fake_app_remoting_report_issue_request.cc", 21 "fake_app_remoting_report_issue_request.cc",
20 "fake_app_remoting_report_issue_request.h", 22 "fake_app_remoting_report_issue_request.h",
21 "fake_network_dispatcher.cc", 23 "fake_network_dispatcher.cc",
22 "fake_network_dispatcher.h", 24 "fake_network_dispatcher.h",
23 "fake_network_manager.cc", 25 "fake_network_manager.cc",
24 "fake_network_manager.h", 26 "fake_network_manager.h",
25 "fake_port_allocator.cc", 27 "fake_port_allocator.cc",
26 "fake_port_allocator.h", 28 "fake_port_allocator.h",
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
118 "//testing/gtest", 120 "//testing/gtest",
119 ] 121 ]
120 } 122 }
121 123
122 source_set("unit_tests") { 124 source_set("unit_tests") {
123 testonly = true 125 testonly = true
124 126
125 sources = [ 127 sources = [
126 "access_token_fetcher_unittest.cc", 128 "access_token_fetcher_unittest.cc",
127 "app_remoting_report_issue_request_unittest.cc", 129 "app_remoting_report_issue_request_unittest.cc",
130 "connection_time_observer_unittest.cc",
128 "host_list_fetcher_unittest.cc", 131 "host_list_fetcher_unittest.cc",
129 "remote_host_info_fetcher_unittest.cc", 132 "remote_host_info_fetcher_unittest.cc",
130 "test_chromoting_client_unittest.cc", 133 "test_chromoting_client_unittest.cc",
131 "test_video_renderer_unittest.cc", 134 "test_video_renderer_unittest.cc",
132 135
133 # TODO(sergeyu): app_remoting_test_driver_environment_unittest.cc 136 # TODO(sergeyu): app_remoting_test_driver_environment_unittest.cc
134 # depends on ar_test_driver_common target and that target implicitly 137 # depends on ar_test_driver_common target and that target implicitly
135 # depends on app_remoting_test_driver_environment_app_details.cc to 138 # depends on app_remoting_test_driver_environment_app_details.cc to
136 # allow some parameters to be overridden (i.e. *app_details.cc file can 139 # allow some parameters to be overridden (i.e. *app_details.cc file can
137 # be replace with a different one). This means that app_deails.cc file 140 # be replace with a different one). This means that app_deails.cc file
138 # has to be included here explicitly. Fix 141 # has to be included here explicitly. Fix
139 # app_remoting_test_driver_environment.cc to avoid this implicit 142 # app_remoting_test_driver_environment.cc to avoid this implicit
140 # dependency on *app_details.cc . 143 # dependency on *app_details.cc .
141 # http://crbug.com/510887 144 # http://crbug.com/510887
142 "app_remoting_test_driver_environment_app_details.cc", 145 "app_remoting_test_driver_environment_app_details.cc",
143 "app_remoting_test_driver_environment_unittest.cc", 146 "app_remoting_test_driver_environment_unittest.cc",
144 ] 147 ]
145 148
146 deps = [ 149 deps = [
147 ":test_support", 150 ":test_support",
148 ":ar_test_driver_common", 151 ":ar_test_driver_common",
149 "//base", 152 "//base",
150 "//net:test_support", 153 "//net:test_support",
151 "//testing/gmock", 154 "//testing/gmock",
152 "//testing/gtest", 155 "//testing/gtest",
153 "//third_party/libyuv", 156 "//third_party/libyuv",
154 "//third_party/webrtc/modules/desktop_capture", 157 "//third_party/webrtc/modules/desktop_capture",
155 ] 158 ]
156 } 159 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698