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

Unified Diff: talk/media/webrtc/webrtcvoiceengine_unittest.cc

Issue 1273363005: Add send transports to individual webrtc::Call streams. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: rebase+comment 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « talk/media/webrtc/webrtcvideoengine2.cc ('k') | webrtc/audio_send_stream.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: talk/media/webrtc/webrtcvoiceengine_unittest.cc
diff --git a/talk/media/webrtc/webrtcvoiceengine_unittest.cc b/talk/media/webrtc/webrtcvoiceengine_unittest.cc
index c559e39f8392af12a4b7a275ad15c011a5da53b5..ef87c74e5814bbccb3ac02fc6f9de3e68e901d51 100644
--- a/talk/media/webrtc/webrtcvoiceengine_unittest.cc
+++ b/talk/media/webrtc/webrtcvoiceengine_unittest.cc
@@ -3354,7 +3354,7 @@ TEST(WebRtcVoiceEngineTest, CoInitialize) {
#endif
TEST_F(WebRtcVoiceEngineTestFake, SetsSyncGroupFromSyncLabel) {
- cricket::FakeCall call(webrtc::Call::Config(nullptr));
+ cricket::FakeCall call((webrtc::Call::Config()));
const uint32 kAudioSsrc = 123;
const std::string kSyncLabel = "AvSyncLabel";
@@ -3384,7 +3384,7 @@ TEST_F(WebRtcVoiceEngineTestFake, SetsSyncGroupFromSyncLabel) {
TEST_F(WebRtcVoiceEngineTestFake, CanChangeCombinedBweOption) {
// Test that changing the combined_audio_video_bwe option results in the
// expected state changes on an associated Call.
- cricket::FakeCall call(webrtc::Call::Config(nullptr));
+ cricket::FakeCall call((webrtc::Call::Config()));
const uint32 kAudioSsrc1 = 223;
const uint32 kAudioSsrc2 = 224;
@@ -3437,8 +3437,8 @@ TEST_F(WebRtcVoiceEngineTestFake, CanChangeCombinedBweOption) {
TEST_F(WebRtcVoiceEngineTestFake, SetCallConfiguresAudioReceiveChannels) {
// Test that calling SetCall() on the voice media channel results in the
// expected state changes in Call.
- cricket::FakeCall call(webrtc::Call::Config(nullptr));
- cricket::FakeCall call2(webrtc::Call::Config(nullptr));
+ cricket::FakeCall call((webrtc::Call::Config()));
+ cricket::FakeCall call2((webrtc::Call::Config()));
const uint32 kAudioSsrc1 = 223;
const uint32 kAudioSsrc2 = 224;
@@ -3474,7 +3474,7 @@ TEST_F(WebRtcVoiceEngineTestFake, SetCallConfiguresAudioReceiveChannels) {
TEST_F(WebRtcVoiceEngineTestFake, ConfigureCombinedBweForNewRecvStreams) {
// Test that adding receive streams after enabling combined bandwidth
// estimation will correctly configure each channel.
- cricket::FakeCall call(webrtc::Call::Config(nullptr));
+ cricket::FakeCall call((webrtc::Call::Config()));
EXPECT_TRUE(SetupEngine());
cricket::WebRtcVoiceMediaChannel* media_channel =
@@ -3502,7 +3502,7 @@ TEST_F(WebRtcVoiceEngineTestFake, ConfigureCombinedBweForNewRecvStreams) {
TEST_F(WebRtcVoiceEngineTestFake, ConfiguresAudioReceiveStreamRtpExtensions) {
// Test that setting the header extensions results in the expected state
// changes on an associated Call.
- cricket::FakeCall call(webrtc::Call::Config(nullptr));
+ cricket::FakeCall call((webrtc::Call::Config()));
std::vector<uint32> ssrcs;
ssrcs.push_back(223);
ssrcs.push_back(224);
@@ -3556,7 +3556,7 @@ TEST_F(WebRtcVoiceEngineTestFake, ConfiguresAudioReceiveStreamRtpExtensions) {
TEST_F(WebRtcVoiceEngineTestFake, DeliverAudioPacket_Call) {
// Test that packets are forwarded to the Call when configured accordingly.
- cricket::FakeCall call(webrtc::Call::Config(nullptr));
+ cricket::FakeCall call((webrtc::Call::Config()));
const uint32 kAudioSsrc = 1;
rtc::Buffer kPcmuPacket(kPcmuFrame, sizeof(kPcmuFrame));
static const unsigned char kRtcp[] = {
« no previous file with comments | « talk/media/webrtc/webrtcvideoengine2.cc ('k') | webrtc/audio_send_stream.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698