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

Unified Diff: webrtc/audio/audio_receive_stream_unittest.cc

Issue 1748403002: Move RtcEventLog object from inside VoiceEngine to Call. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Undid unneccessary changes to rtp_rtcp module. Created 4 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 side-by-side diff with in-line comments
Download patch
Index: webrtc/audio/audio_receive_stream_unittest.cc
diff --git a/webrtc/audio/audio_receive_stream_unittest.cc b/webrtc/audio/audio_receive_stream_unittest.cc
index 8703d6ed324819a0a041e823a88bf90db6f9e7ca..c39d7ed97a6ef316d973ff0d14d436d5358df0fa 100644
--- a/webrtc/audio/audio_receive_stream_unittest.cc
+++ b/webrtc/audio/audio_receive_stream_unittest.cc
@@ -66,7 +66,8 @@ struct ConfigHelper {
: simulated_clock_(123456),
congestion_controller_(&simulated_clock_,
&bitrate_observer_,
- &remote_bitrate_observer_) {
+ &remote_bitrate_observer_,
+ nullptr) {
using testing::Invoke;
EXPECT_CALL(voice_engine_,
@@ -220,8 +221,9 @@ TEST(AudioReceiveStreamTest, ConfigToString) {
TEST(AudioReceiveStreamTest, ConstructDestruct) {
ConfigHelper helper;
- internal::AudioReceiveStream recv_stream(
- helper.congestion_controller(), helper.config(), helper.audio_state());
+ internal::AudioReceiveStream recv_stream(helper.congestion_controller(),
+ helper.config(),
+ helper.audio_state(), nullptr);
}
MATCHER_P(VerifyHeaderExtension, expected_extension, "") {
@@ -239,8 +241,9 @@ TEST(AudioReceiveStreamTest, AudioPacketUpdatesBweFeedback) {
ConfigHelper helper;
helper.config().rtp.transport_cc = true;
helper.SetupMockForBweFeedback(true);
- internal::AudioReceiveStream recv_stream(
- helper.congestion_controller(), helper.config(), helper.audio_state());
+ internal::AudioReceiveStream recv_stream(helper.congestion_controller(),
+ helper.config(),
+ helper.audio_state(), nullptr);
const int kTransportSequenceNumberValue = 1234;
std::vector<uint8_t> rtp_packet = CreateRtpHeaderWithOneByteExtension(
kTransportSequenceNumberId, kTransportSequenceNumberValue, 2);
@@ -260,8 +263,9 @@ TEST(AudioReceiveStreamTest, AudioPacketUpdatesBweFeedback) {
TEST(AudioReceiveStreamTest, GetStats) {
ConfigHelper helper;
- internal::AudioReceiveStream recv_stream(
- helper.congestion_controller(), helper.config(), helper.audio_state());
+ internal::AudioReceiveStream recv_stream(helper.congestion_controller(),
+ helper.config(),
+ helper.audio_state(), nullptr);
helper.SetupMockForGetStats();
AudioReceiveStream::Stats stats = recv_stream.GetStats();
EXPECT_EQ(kRemoteSsrc, stats.remote_ssrc);

Powered by Google App Engine
This is Rietveld 408576698