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

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

Issue 2531043002: Reland of move RTPPayloadStrategy and simplify RTPPayloadRegistry (Closed)
Patch Set: Remove deprecated RegisterReceivePayload function in RtpReceiver Created 4 years 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 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 } 94 }
95 ~RtpRtcpAPITest() {} 95 ~RtpRtcpAPITest() {}
96 96
97 void SetUp() override { 97 void SetUp() override {
98 RtpRtcp::Configuration configuration; 98 RtpRtcp::Configuration configuration;
99 configuration.audio = true; 99 configuration.audio = true;
100 configuration.clock = &fake_clock_; 100 configuration.clock = &fake_clock_;
101 configuration.outgoing_transport = &null_transport_; 101 configuration.outgoing_transport = &null_transport_;
102 configuration.retransmission_rate_limiter = &retransmission_rate_limiter_; 102 configuration.retransmission_rate_limiter = &retransmission_rate_limiter_;
103 module_.reset(RtpRtcp::CreateRtpRtcp(configuration)); 103 module_.reset(RtpRtcp::CreateRtpRtcp(configuration));
104 rtp_payload_registry_.reset(new RTPPayloadRegistry( 104 rtp_payload_registry_.reset(new RTPPayloadRegistry());
105 RTPPayloadStrategy::CreateStrategy(true)));
106 rtp_receiver_.reset(RtpReceiver::CreateAudioReceiver( 105 rtp_receiver_.reset(RtpReceiver::CreateAudioReceiver(
107 &fake_clock_, NULL, NULL, rtp_payload_registry_.get())); 106 &fake_clock_, NULL, NULL, rtp_payload_registry_.get()));
108 } 107 }
109 108
110 std::unique_ptr<RTPPayloadRegistry> rtp_payload_registry_; 109 std::unique_ptr<RTPPayloadRegistry> rtp_payload_registry_;
111 std::unique_ptr<RtpReceiver> rtp_receiver_; 110 std::unique_ptr<RtpReceiver> rtp_receiver_;
112 std::unique_ptr<RtpRtcp> module_; 111 std::unique_ptr<RtpRtcp> module_;
113 uint32_t test_ssrc_; 112 uint32_t test_ssrc_;
114 uint32_t test_timestamp_; 113 uint32_t test_timestamp_;
115 uint16_t test_sequence_number_; 114 uint16_t test_sequence_number_;
(...skipping 74 matching lines...) Expand 10 before | Expand all | Expand 10 after
190 rtx_header.payloadType = kRtxPayloadType; 189 rtx_header.payloadType = kRtxPayloadType;
191 EXPECT_TRUE(rtp_payload_registry_->IsRtx(rtx_header)); 190 EXPECT_TRUE(rtp_payload_registry_->IsRtx(rtx_header));
192 rtx_header.ssrc = 0; 191 rtx_header.ssrc = 0;
193 EXPECT_FALSE(rtp_payload_registry_->IsRtx(rtx_header)); 192 EXPECT_FALSE(rtp_payload_registry_->IsRtx(rtx_header));
194 rtx_header.ssrc = kRtxSsrc; 193 rtx_header.ssrc = kRtxSsrc;
195 rtx_header.payloadType = 0; 194 rtx_header.payloadType = 0;
196 EXPECT_TRUE(rtp_payload_registry_->IsRtx(rtx_header)); 195 EXPECT_TRUE(rtp_payload_registry_->IsRtx(rtx_header));
197 } 196 }
198 197
199 } // namespace webrtc 198 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtp_utility.h ('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