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

Side by Side Diff: webrtc/media/engine/webrtcvoiceengine_unittest.cc

Issue 2501503004: Remove Absolute Send Time from list of supported header extensions for audio streams. (Closed)
Patch Set: test fix Created 4 years, 1 month 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 | « webrtc/config.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 /* 1 /*
2 * Copyright (c) 2008 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2008 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 10
(...skipping 2032 matching lines...) Expand 10 before | Expand all | Expand 10 after
2043 } 2043 }
2044 2044
2045 // Test support for audio level header extension. 2045 // Test support for audio level header extension.
2046 TEST_F(WebRtcVoiceEngineTestFake, SendAudioLevelHeaderExtensions) { 2046 TEST_F(WebRtcVoiceEngineTestFake, SendAudioLevelHeaderExtensions) {
2047 TestSetSendRtpHeaderExtensions(webrtc::RtpExtension::kAudioLevelUri); 2047 TestSetSendRtpHeaderExtensions(webrtc::RtpExtension::kAudioLevelUri);
2048 } 2048 }
2049 TEST_F(WebRtcVoiceEngineTestFake, RecvAudioLevelHeaderExtensions) { 2049 TEST_F(WebRtcVoiceEngineTestFake, RecvAudioLevelHeaderExtensions) {
2050 TestSetRecvRtpHeaderExtensions(webrtc::RtpExtension::kAudioLevelUri); 2050 TestSetRecvRtpHeaderExtensions(webrtc::RtpExtension::kAudioLevelUri);
2051 } 2051 }
2052 2052
2053 // Test support for absolute send time header extension. 2053 // Test support for transport sequence number header extension.
2054 TEST_F(WebRtcVoiceEngineTestFake, SendAbsoluteSendTimeHeaderExtensions) { 2054 TEST_F(WebRtcVoiceEngineTestFake, SendTransportSequenceNumberHeaderExtensions) {
2055 TestSetSendRtpHeaderExtensions(webrtc::RtpExtension::kAbsSendTimeUri); 2055 TestSetSendRtpHeaderExtensions(
2056 webrtc::RtpExtension::kTransportSequenceNumberUri);
2056 } 2057 }
2057 TEST_F(WebRtcVoiceEngineTestFake, RecvAbsoluteSendTimeHeaderExtensions) { 2058 TEST_F(WebRtcVoiceEngineTestFake, RecvTransportSequenceNumberHeaderExtensions) {
2058 TestSetRecvRtpHeaderExtensions(webrtc::RtpExtension::kAbsSendTimeUri); 2059 TestSetRecvRtpHeaderExtensions(
2060 webrtc::RtpExtension::kTransportSequenceNumberUri);
2059 } 2061 }
2060 2062
2061 // Test that we can create a channel and start sending on it. 2063 // Test that we can create a channel and start sending on it.
2062 TEST_F(WebRtcVoiceEngineTestFake, Send) { 2064 TEST_F(WebRtcVoiceEngineTestFake, Send) {
2063 EXPECT_TRUE(SetupSendStream()); 2065 EXPECT_TRUE(SetupSendStream());
2064 SetSendParameters(send_parameters_); 2066 SetSendParameters(send_parameters_);
2065 SetSend(true); 2067 SetSend(true);
2066 EXPECT_TRUE(GetSendStream(kSsrc1).IsSending()); 2068 EXPECT_TRUE(GetSendStream(kSsrc1).IsSending());
2067 SetSend(false); 2069 SetSend(false);
2068 EXPECT_FALSE(GetSendStream(kSsrc1).IsSending()); 2070 EXPECT_FALSE(GetSendStream(kSsrc1).IsSending());
(...skipping 1428 matching lines...) Expand 10 before | Expand all | Expand 10 after
3497 nullptr, webrtc::CreateBuiltinAudioDecoderFactory()); 3499 nullptr, webrtc::CreateBuiltinAudioDecoderFactory());
3498 webrtc::RtcEventLogNullImpl event_log; 3500 webrtc::RtcEventLogNullImpl event_log;
3499 std::unique_ptr<webrtc::Call> call( 3501 std::unique_ptr<webrtc::Call> call(
3500 webrtc::Call::Create(webrtc::Call::Config(&event_log))); 3502 webrtc::Call::Create(webrtc::Call::Config(&event_log)));
3501 cricket::WebRtcVoiceMediaChannel channel(&engine, cricket::MediaConfig(), 3503 cricket::WebRtcVoiceMediaChannel channel(&engine, cricket::MediaConfig(),
3502 cricket::AudioOptions(), call.get()); 3504 cricket::AudioOptions(), call.get());
3503 cricket::AudioRecvParameters parameters; 3505 cricket::AudioRecvParameters parameters;
3504 parameters.codecs = engine.recv_codecs(); 3506 parameters.codecs = engine.recv_codecs();
3505 EXPECT_TRUE(channel.SetRecvParameters(parameters)); 3507 EXPECT_TRUE(channel.SetRecvParameters(parameters));
3506 } 3508 }
OLDNEW
« no previous file with comments | « webrtc/config.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698