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

Side by Side Diff: webrtc/voice_engine/channel.cc

Issue 2177263002: Regression test for issue where Opus DTX status was being forgotten. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Initial version. Created 4 years, 5 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) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 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 1530 matching lines...) Expand 10 before | Expand all | Expand 10 after
1541 int ret = enable_dtx ? audio_coding_->EnableOpusDtx() 1541 int ret = enable_dtx ? audio_coding_->EnableOpusDtx()
1542 : audio_coding_->DisableOpusDtx(); 1542 : audio_coding_->DisableOpusDtx();
1543 if (ret != 0) { 1543 if (ret != 0) {
1544 _engineStatisticsPtr->SetLastError(VE_AUDIO_CODING_MODULE_ERROR, 1544 _engineStatisticsPtr->SetLastError(VE_AUDIO_CODING_MODULE_ERROR,
1545 kTraceError, "SetOpusDtx() failed"); 1545 kTraceError, "SetOpusDtx() failed");
1546 return -1; 1546 return -1;
1547 } 1547 }
1548 return 0; 1548 return 0;
1549 } 1549 }
1550 1550
1551 int Channel::GetOpusDtx(bool* enabled) {
1552 return audio_coding_->GetOpusDtx(enabled);
1553 }
1554
1551 int32_t Channel::RegisterExternalTransport(Transport* transport) { 1555 int32_t Channel::RegisterExternalTransport(Transport* transport) {
1552 WEBRTC_TRACE(kTraceInfo, kTraceVoice, VoEId(_instanceId, _channelId), 1556 WEBRTC_TRACE(kTraceInfo, kTraceVoice, VoEId(_instanceId, _channelId),
1553 "Channel::RegisterExternalTransport()"); 1557 "Channel::RegisterExternalTransport()");
1554 1558
1555 rtc::CritScope cs(&_callbackCritSect); 1559 rtc::CritScope cs(&_callbackCritSect);
1556 if (_externalTransport) { 1560 if (_externalTransport) {
1557 _engineStatisticsPtr->SetLastError( 1561 _engineStatisticsPtr->SetLastError(
1558 VE_INVALID_OPERATION, kTraceError, 1562 VE_INVALID_OPERATION, kTraceError,
1559 "RegisterExternalTransport() external transport already enabled"); 1563 "RegisterExternalTransport() external transport already enabled");
1560 return -1; 1564 return -1;
(...skipping 2001 matching lines...) Expand 10 before | Expand all | Expand 10 after
3562 int64_t min_rtt = 0; 3566 int64_t min_rtt = 0;
3563 if (_rtpRtcpModule->RTT(remoteSSRC, &rtt, &avg_rtt, &min_rtt, &max_rtt) != 3567 if (_rtpRtcpModule->RTT(remoteSSRC, &rtt, &avg_rtt, &min_rtt, &max_rtt) !=
3564 0) { 3568 0) {
3565 return 0; 3569 return 0;
3566 } 3570 }
3567 return rtt; 3571 return rtt;
3568 } 3572 }
3569 3573
3570 } // namespace voe 3574 } // namespace voe
3571 } // namespace webrtc 3575 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698