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

Side by Side Diff: webrtc/video/video_quality_test.cc

Issue 2717973005: Test field trial group with startswith rather than equals. (Closed)
Patch Set: Added IsEnabled() convenience function Created 3 years, 9 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 /* 1 /*
2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 #include "webrtc/video/video_quality_test.h" 10 #include "webrtc/video/video_quality_test.h"
(...skipping 1423 matching lines...) Expand 10 before | Expand all | Expand 10 after
1434 if (graph_title.empty()) 1434 if (graph_title.empty())
1435 graph_title = VideoQualityTest::GenerateGraphTitle(); 1435 graph_title = VideoQualityTest::GenerateGraphTitle();
1436 1436
1437 // In the case of different resolutions, the functions calculating PSNR and 1437 // In the case of different resolutions, the functions calculating PSNR and
1438 // SSIM return -1.0, instead of a positive value as usual. VideoAnalyzer 1438 // SSIM return -1.0, instead of a positive value as usual. VideoAnalyzer
1439 // aborts if the average psnr/ssim are below the given threshold, which is 1439 // aborts if the average psnr/ssim are below the given threshold, which is
1440 // 0.0 by default. Setting the thresholds to -1.1 prevents the unnecessary 1440 // 0.0 by default. Setting the thresholds to -1.1 prevents the unnecessary
1441 // abort. 1441 // abort.
1442 VideoStream& selected_stream = params_.ss.streams[params_.ss.selected_stream]; 1442 VideoStream& selected_stream = params_.ss.streams[params_.ss.selected_stream];
1443 1443
1444 bool is_quick_test_enabled = 1444 bool is_quick_test_enabled = field_trial::IsEnabled("WebRTC-QuickPerfTest");
1445 field_trial::FindFullName("WebRTC-QuickPerfTest") == "Enabled";
1446 VideoAnalyzer analyzer( 1445 VideoAnalyzer analyzer(
1447 &send_transport, params_.analyzer.test_label, 1446 &send_transport, params_.analyzer.test_label,
1448 1447
1449 params_.analyzer.avg_psnr_threshold, params_.analyzer.avg_ssim_threshold, 1448 params_.analyzer.avg_psnr_threshold, params_.analyzer.avg_ssim_threshold,
1450 is_quick_test_enabled 1449 is_quick_test_enabled
1451 ? kFramesSentInQuickTest 1450 ? kFramesSentInQuickTest
1452 : params_.analyzer.test_durations_secs * params_.video.fps, 1451 : params_.analyzer.test_durations_secs * params_.video.fps,
1453 graph_data_output_file, graph_title, 1452 graph_data_output_file, graph_title,
1454 kVideoSendSsrcs[params_.ss.selected_stream], 1453 kVideoSendSsrcs[params_.ss.selected_stream],
1455 kSendRtxSsrcs[params_.ss.selected_stream], 1454 kSendRtxSsrcs[params_.ss.selected_stream],
(...skipping 245 matching lines...) Expand 10 before | Expand all | Expand 10 after
1701 std::ostringstream str; 1700 std::ostringstream str;
1702 str << receive_logs_++; 1701 str << receive_logs_++;
1703 std::string path = 1702 std::string path =
1704 params_.video.encoded_frame_base_path + "." + str.str() + ".recv.ivf"; 1703 params_.video.encoded_frame_base_path + "." + str.str() + ".recv.ivf";
1705 stream->EnableEncodedFrameRecording(rtc::CreatePlatformFile(path), 1704 stream->EnableEncodedFrameRecording(rtc::CreatePlatformFile(path),
1706 10000000); 1705 10000000);
1707 } 1706 }
1708 } 1707 }
1709 1708
1710 } // namespace webrtc 1709 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698