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

Side by Side Diff: chrome/browser/media/webrtc_perf_browsertest.cc

Issue 1839653003: Filter send-only and recv-only as appropriate in WebRTC perf tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed bool return values Created 4 years, 8 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
« no previous file with comments | « chrome/browser/media/webrtc_browsertest_perf.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #include "base/command_line.h" 5 #include "base/command_line.h"
6 #include "base/files/file_util.h" 6 #include "base/files/file_util.h"
7 #include "base/json/json_reader.h" 7 #include "base/json/json_reader.h"
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "base/strings/string_split.h" 10 #include "base/strings/string_split.h"
(...skipping 165 matching lines...) Expand 10 before | Expand all | Expand 10 after
176 scoped_ptr<base::DictionaryValue> all_data = 176 scoped_ptr<base::DictionaryValue> all_data =
177 MeasureWebRtcInternalsData(10000); 177 MeasureWebRtcInternalsData(10000);
178 ASSERT_TRUE(all_data.get() != NULL); 178 ASSERT_TRUE(all_data.get() != NULL);
179 179
180 // This assumes the sending peer connection is always listed first in the 180 // This assumes the sending peer connection is always listed first in the
181 // data store, and the receiving second. 181 // data store, and the receiving second.
182 const base::DictionaryValue* first_pc_dict = 182 const base::DictionaryValue* first_pc_dict =
183 GetDataOnPeerConnection(all_data.get(), 0); 183 GetDataOnPeerConnection(all_data.get(), 0);
184 ASSERT_TRUE(first_pc_dict != NULL); 184 ASSERT_TRUE(first_pc_dict != NULL);
185 test::PrintBweForVideoMetrics(*first_pc_dict, "_sendonly", video_codec); 185 test::PrintBweForVideoMetrics(*first_pc_dict, "_sendonly", video_codec);
186 test::PrintMetricsForAllStreams(*first_pc_dict, "_sendonly", video_codec); 186 test::PrintMetricsForSendStreams(*first_pc_dict, "_sendonly", video_codec);
187 187
188 const base::DictionaryValue* second_pc_dict = 188 const base::DictionaryValue* second_pc_dict =
189 GetDataOnPeerConnection(all_data.get(), 1); 189 GetDataOnPeerConnection(all_data.get(), 1);
190 ASSERT_TRUE(second_pc_dict != NULL); 190 ASSERT_TRUE(second_pc_dict != NULL);
191 test::PrintBweForVideoMetrics(*second_pc_dict, "_recvonly", video_codec); 191 test::PrintBweForVideoMetrics(*second_pc_dict, "_recvonly", video_codec);
192 test::PrintMetricsForAllStreams(*second_pc_dict, "_recvonly", video_codec); 192 test::PrintMetricsForRecvStreams(*second_pc_dict, "_recvonly", video_codec);
193 193
194 HangUp(left_tab); 194 HangUp(left_tab);
195 HangUp(right_tab); 195 HangUp(right_tab);
196 } 196 }
197 }; 197 };
198 198
199 // This is manual for its long execution time. 199 // This is manual for its long execution time.
200 200
201 // The video codec name is now appended to result bucket (e.g. 'video_tx_VP8'). 201 // The video codec name is now appended to result bucket (e.g. 'video_tx_VP8').
202 // TODO(asapersson): Keep test below using the default video codec (which do 202 // TODO(asapersson): Keep test below using the default video codec (which do
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
236 } 236 }
237 237
238 #endif // BUILDFLAG(RTC_USE_H264) 238 #endif // BUILDFLAG(RTC_USE_H264)
239 239
240 IN_PROC_BROWSER_TEST_F( 240 IN_PROC_BROWSER_TEST_F(
241 WebRtcPerfBrowserTest, 241 WebRtcPerfBrowserTest,
242 MANUAL_RunsOneWayCall60SecsAndLogsInternalMetricsDefault) { 242 MANUAL_RunsOneWayCall60SecsAndLogsInternalMetricsDefault) {
243 RunsOneWayCall60SecsAndLogsInternalMetrics(""); 243 RunsOneWayCall60SecsAndLogsInternalMetrics("");
244 } 244 }
245 245
OLDNEW
« no previous file with comments | « chrome/browser/media/webrtc_browsertest_perf.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698