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

Unified Diff: webrtc/video/video_quality_test.cc

Issue 2353033005: Refactoring: move ownership of RtcEventLog from Call to PeerConnection (Closed)
Patch Set: Updated unit tests to use RtcEventLogNullImpl. Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: webrtc/video/video_quality_test.cc
diff --git a/webrtc/video/video_quality_test.cc b/webrtc/video/video_quality_test.cc
index 4ac86d1a1521a929675cda583432086793951ff1..bb6f1a5a75a3c18d9cd892ca45fa0b16d26cbfc5 100644
--- a/webrtc/video/video_quality_test.cc
+++ b/webrtc/video/video_quality_test.cc
@@ -25,6 +25,7 @@
#include "webrtc/base/timeutils.h"
#include "webrtc/call.h"
#include "webrtc/common_video/libyuv/include/webrtc_libyuv.h"
+#include "webrtc/logging/rtc_event_log/rtc_event_log.h"
#include "webrtc/modules/rtp_rtcp/include/rtp_header_parser.h"
#include "webrtc/modules/rtp_rtcp/source/rtp_utility.h"
#include "webrtc/system_wrappers/include/cpu_info.h"
@@ -1151,7 +1152,9 @@ void VideoQualityTest::RunWithAnalyzer(const Params& params) {
<< "!";
}
+ webrtc::RtcEventLogNullImpl event_log;
Call::Config call_config;
+ call_config.event_log = &event_log;
call_config.bitrate_config = params.common.call_bitrate_config;
CreateCalls(call_config, call_config);
@@ -1261,7 +1264,9 @@ void VideoQualityTest::RunWithRenderers(const Params& params) {
// TODO(ivica): Remove bitrate_config and use the default Call::Config(), to
// match the full stack tests.
+ webrtc::RtcEventLogNullImpl event_log;
Call::Config call_config;
+ call_config.event_log = &event_log;
call_config.bitrate_config = params_.common.call_bitrate_config;
::VoiceEngineState voe;

Powered by Google App Engine
This is Rietveld 408576698