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

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: Removed timer from observer, used print friendly string in TestChromotingClient, and made modificat… 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 "chromoting_test_driver_environment.cc", 15 "chromoting_test_driver_environment.cc",
16 "chromoting_test_driver_environment.h", 16 "chromoting_test_driver_environment.h",
17 "connection_setup_info.cc", 17 "connection_setup_info.cc",
18 "connection_setup_info.h", 18 "connection_setup_info.h",
19 "connection_time_observer.cc",
20 "connection_time_observer.h",
19 "fake_access_token_fetcher.cc", 21 "fake_access_token_fetcher.cc",
20 "fake_access_token_fetcher.h", 22 "fake_access_token_fetcher.h",
21 "fake_app_remoting_report_issue_request.cc", 23 "fake_app_remoting_report_issue_request.cc",
22 "fake_app_remoting_report_issue_request.h", 24 "fake_app_remoting_report_issue_request.h",
23 "fake_host_list_fetcher.cc", 25 "fake_host_list_fetcher.cc",
24 "fake_host_list_fetcher.h", 26 "fake_host_list_fetcher.h",
25 "fake_network_dispatcher.cc", 27 "fake_network_dispatcher.cc",
26 "fake_network_dispatcher.h", 28 "fake_network_dispatcher.h",
27 "fake_network_manager.cc", 29 "fake_network_manager.cc",
28 "fake_network_manager.h", 30 "fake_network_manager.h",
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
125 ] 127 ]
126 } 128 }
127 129
128 source_set("unit_tests") { 130 source_set("unit_tests") {
129 testonly = true 131 testonly = true
130 132
131 sources = [ 133 sources = [
132 "access_token_fetcher_unittest.cc", 134 "access_token_fetcher_unittest.cc",
133 "app_remoting_report_issue_request_unittest.cc", 135 "app_remoting_report_issue_request_unittest.cc",
134 "chromoting_test_driver_environment_unittest.cc", 136 "chromoting_test_driver_environment_unittest.cc",
137 "connection_time_observer_unittest.cc",
135 "host_list_fetcher_unittest.cc", 138 "host_list_fetcher_unittest.cc",
136 "remote_host_info_fetcher_unittest.cc", 139 "remote_host_info_fetcher_unittest.cc",
137 "test_chromoting_client_unittest.cc", 140 "test_chromoting_client_unittest.cc",
138 "test_video_renderer_unittest.cc", 141 "test_video_renderer_unittest.cc",
139 142
140 # TODO(sergeyu): app_remoting_test_driver_environment_unittest.cc 143 # TODO(sergeyu): app_remoting_test_driver_environment_unittest.cc
141 # depends on ar_test_driver_common target and that target implicitly 144 # depends on ar_test_driver_common target and that target implicitly
142 # depends on app_remoting_test_driver_environment_app_details.cc to 145 # depends on app_remoting_test_driver_environment_app_details.cc to
143 # allow some parameters to be overridden (i.e. *app_details.cc file can 146 # allow some parameters to be overridden (i.e. *app_details.cc file can
144 # be replace with a different one). This means that app_deails.cc file 147 # be replace with a different one). This means that app_deails.cc file
145 # has to be included here explicitly. Fix 148 # has to be included here explicitly. Fix
146 # app_remoting_test_driver_environment.cc to avoid this implicit 149 # app_remoting_test_driver_environment.cc to avoid this implicit
147 # dependency on *app_details.cc . 150 # dependency on *app_details.cc .
148 # http://crbug.com/510887 151 # http://crbug.com/510887
149 "app_remoting_test_driver_environment_app_details.cc", 152 "app_remoting_test_driver_environment_app_details.cc",
150 "app_remoting_test_driver_environment_unittest.cc", 153 "app_remoting_test_driver_environment_unittest.cc",
151 ] 154 ]
152 155
153 deps = [ 156 deps = [
154 ":test_support", 157 ":test_support",
155 ":ar_test_driver_common", 158 ":ar_test_driver_common",
156 "//base", 159 "//base",
157 "//net:test_support", 160 "//net:test_support",
158 "//testing/gmock", 161 "//testing/gmock",
159 "//testing/gtest", 162 "//testing/gtest",
160 "//third_party/libyuv", 163 "//third_party/libyuv",
161 "//third_party/webrtc/modules/desktop_capture", 164 "//third_party/webrtc/modules/desktop_capture",
162 ] 165 ]
163 } 166 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698