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

Side by Side Diff: webrtc/modules/rtp_rtcp/test/testAPI/test_api.cc

Issue 1802993002: Clean away use of RtpAudioFeedback interface from RTP/RTCP receiver code. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@voe_dtmf_7
Patch Set: Reanimate CreateAudioReceiver() with 5 params, to not break downstream code. Created 4 years, 8 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 79 matching lines...) Expand 10 before | Expand all | Expand 10 after
90 90
91 void SetUp() override { 91 void SetUp() override {
92 RtpRtcp::Configuration configuration; 92 RtpRtcp::Configuration configuration;
93 configuration.audio = true; 93 configuration.audio = true;
94 configuration.clock = &fake_clock_; 94 configuration.clock = &fake_clock_;
95 configuration.outgoing_transport = &null_transport_; 95 configuration.outgoing_transport = &null_transport_;
96 module_.reset(RtpRtcp::CreateRtpRtcp(configuration)); 96 module_.reset(RtpRtcp::CreateRtpRtcp(configuration));
97 rtp_payload_registry_.reset(new RTPPayloadRegistry( 97 rtp_payload_registry_.reset(new RTPPayloadRegistry(
98 RTPPayloadStrategy::CreateStrategy(true))); 98 RTPPayloadStrategy::CreateStrategy(true)));
99 rtp_receiver_.reset(RtpReceiver::CreateAudioReceiver( 99 rtp_receiver_.reset(RtpReceiver::CreateAudioReceiver(
100 &fake_clock_, NULL, NULL, NULL, rtp_payload_registry_.get())); 100 &fake_clock_, NULL, NULL, rtp_payload_registry_.get()));
101 } 101 }
102 102
103 rtc::scoped_ptr<RTPPayloadRegistry> rtp_payload_registry_; 103 rtc::scoped_ptr<RTPPayloadRegistry> rtp_payload_registry_;
104 rtc::scoped_ptr<RtpReceiver> rtp_receiver_; 104 rtc::scoped_ptr<RtpReceiver> rtp_receiver_;
105 rtc::scoped_ptr<RtpRtcp> module_; 105 rtc::scoped_ptr<RtpRtcp> module_;
106 uint32_t test_ssrc_; 106 uint32_t test_ssrc_;
107 uint32_t test_timestamp_; 107 uint32_t test_timestamp_;
108 uint16_t test_sequence_number_; 108 uint16_t test_sequence_number_;
109 std::vector<uint32_t> test_csrcs_; 109 std::vector<uint32_t> test_csrcs_;
110 SimulatedClock fake_clock_; 110 SimulatedClock fake_clock_;
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
181 rtx_header.payloadType = kRtxPayloadType; 181 rtx_header.payloadType = kRtxPayloadType;
182 EXPECT_TRUE(rtp_payload_registry_->IsRtx(rtx_header)); 182 EXPECT_TRUE(rtp_payload_registry_->IsRtx(rtx_header));
183 rtx_header.ssrc = 0; 183 rtx_header.ssrc = 0;
184 EXPECT_FALSE(rtp_payload_registry_->IsRtx(rtx_header)); 184 EXPECT_FALSE(rtp_payload_registry_->IsRtx(rtx_header));
185 rtx_header.ssrc = kRtxSsrc; 185 rtx_header.ssrc = kRtxSsrc;
186 rtx_header.payloadType = 0; 186 rtx_header.payloadType = 0;
187 EXPECT_TRUE(rtp_payload_registry_->IsRtx(rtx_header)); 187 EXPECT_TRUE(rtp_payload_registry_->IsRtx(rtx_header));
188 } 188 }
189 189
190 } // namespace webrtc 190 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtp_utility.cc ('k') | webrtc/modules/rtp_rtcp/test/testAPI/test_api_audio.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698