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

Side by Side Diff: webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl_unittest.cc

Issue 2986893002: Piggybacking simulcast id and ALR experiment id into video content type extension. (Closed)
Patch Set: Remove static initialization from VideoContentType and remove memoization in AlrDetector Created 3 years, 3 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) 2013 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2013 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 213 matching lines...) Expand 10 before | Expand all | Expand 10 after
224 RtpRtcpModule receiver_; 224 RtpRtcpModule receiver_;
225 VideoCodec codec_; 225 VideoCodec codec_;
226 226
227 void SendFrame(const RtpRtcpModule* module, uint8_t tid) { 227 void SendFrame(const RtpRtcpModule* module, uint8_t tid) {
228 RTPVideoHeaderVP8 vp8_header = {}; 228 RTPVideoHeaderVP8 vp8_header = {};
229 vp8_header.temporalIdx = tid; 229 vp8_header.temporalIdx = tid;
230 RTPVideoHeader rtp_video_header; 230 RTPVideoHeader rtp_video_header;
231 rtp_video_header.width = codec_.width; 231 rtp_video_header.width = codec_.width;
232 rtp_video_header.height = codec_.height; 232 rtp_video_header.height = codec_.height;
233 rtp_video_header.rotation = kVideoRotation_0; 233 rtp_video_header.rotation = kVideoRotation_0;
234 rtp_video_header.content_type = VideoContentType::UNSPECIFIED; 234 rtp_video_header.content_type = VideoContentType::Unspecified();
235 rtp_video_header.playout_delay = {-1, -1}; 235 rtp_video_header.playout_delay = {-1, -1};
236 rtp_video_header.is_first_packet_in_frame = true; 236 rtp_video_header.is_first_packet_in_frame = true;
237 rtp_video_header.simulcastIdx = 0; 237 rtp_video_header.simulcastIdx = 0;
238 rtp_video_header.codec = kRtpVideoVp8; 238 rtp_video_header.codec = kRtpVideoVp8;
239 rtp_video_header.codecHeader = {vp8_header}; 239 rtp_video_header.codecHeader = {vp8_header};
240 rtp_video_header.video_timing = {0u, 0u, 0u, 0u, 0u, 0u, false}; 240 rtp_video_header.video_timing = {0u, 0u, 0u, 0u, 0u, 0u, false};
241 241
242 const uint8_t payload[100] = {0}; 242 const uint8_t payload[100] = {0};
243 EXPECT_EQ(true, module->impl_->SendOutgoingData( 243 EXPECT_EQ(true, module->impl_->SendOutgoingData(
244 kVideoFrameKey, codec_.plType, 0, 0, payload, 244 kVideoFrameKey, codec_.plType, 0, 0, payload,
(...skipping 393 matching lines...) Expand 10 before | Expand all | Expand 10 after
638 clock_.AdvanceTimeMilliseconds(kTimeoutMs - 1); 638 clock_.AdvanceTimeMilliseconds(kTimeoutMs - 1);
639 sender_.impl_->Process(); 639 sender_.impl_->Process();
640 EXPECT_EQ(2U, sender_.transport_.NumKeepaliveSent()); 640 EXPECT_EQ(2U, sender_.transport_.NumKeepaliveSent());
641 641
642 // Timeout relative payload, send new keep-alive. 642 // Timeout relative payload, send new keep-alive.
643 clock_.AdvanceTimeMilliseconds(1); 643 clock_.AdvanceTimeMilliseconds(1);
644 sender_.impl_->Process(); 644 sender_.impl_->Process();
645 EXPECT_EQ(3U, sender_.transport_.NumKeepaliveSent()); 645 EXPECT_EQ(3U, sender_.transport_.NumKeepaliveSent());
646 } 646 }
647 } // namespace webrtc 647 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698