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

Side by Side Diff: webrtc/call/flexfec_receive_stream_unittest.cc

Issue 2625633003: Let FlexfecReceiveStreamImpl send RTCP RRs. (Closed)
Patch Set: Rebase. Created 3 years, 11 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
« no previous file with comments | « webrtc/call/flexfec_receive_stream_impl.cc ('k') | webrtc/media/engine/webrtcvideoengine2.cc » ('j') | 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) 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2016 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
11 #include <stdint.h> 11 #include <stdint.h>
12 12
13 #include "webrtc/base/array_view.h" 13 #include "webrtc/base/array_view.h"
14 #include "webrtc/call/flexfec_receive_stream_impl.h" 14 #include "webrtc/call/flexfec_receive_stream_impl.h"
15 #include "webrtc/modules/pacing/packet_router.h"
15 #include "webrtc/modules/rtp_rtcp/include/flexfec_receiver.h" 16 #include "webrtc/modules/rtp_rtcp/include/flexfec_receiver.h"
16 #include "webrtc/modules/rtp_rtcp/mocks/mock_recovered_packet_receiver.h" 17 #include "webrtc/modules/rtp_rtcp/mocks/mock_recovered_packet_receiver.h"
18 #include "webrtc/modules/rtp_rtcp/mocks/mock_rtcp_rtt_stats.h"
17 #include "webrtc/modules/rtp_rtcp/source/byte_io.h" 19 #include "webrtc/modules/rtp_rtcp/source/byte_io.h"
18 #include "webrtc/modules/rtp_rtcp/source/rtp_header_extensions.h" 20 #include "webrtc/modules/rtp_rtcp/source/rtp_header_extensions.h"
21 #include "webrtc/modules/utility/include/mock/mock_process_thread.h"
19 #include "webrtc/test/gmock.h" 22 #include "webrtc/test/gmock.h"
20 #include "webrtc/test/gtest.h" 23 #include "webrtc/test/gtest.h"
21 #include "webrtc/test/mock_transport.h" 24 #include "webrtc/test/mock_transport.h"
22 25
23 namespace webrtc { 26 namespace webrtc {
24 27
25 namespace { 28 namespace {
26 29
27 constexpr uint8_t kFlexfecPlType = 118; 30 constexpr uint8_t kFlexfecPlType = 118;
28 constexpr uint8_t kFlexfecSsrc[] = {0x00, 0x00, 0x00, 0x01}; 31 constexpr uint8_t kFlexfecSsrc[] = {0x00, 0x00, 0x00, 0x01};
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
67 EXPECT_TRUE(config.IsCompleteAndEnabled()); 70 EXPECT_TRUE(config.IsCompleteAndEnabled());
68 71
69 config.protected_media_ssrcs.push_back(33423423); 72 config.protected_media_ssrcs.push_back(33423423);
70 EXPECT_FALSE(config.IsCompleteAndEnabled()); 73 EXPECT_FALSE(config.IsCompleteAndEnabled());
71 } 74 }
72 75
73 class FlexfecReceiveStreamTest : public ::testing::Test { 76 class FlexfecReceiveStreamTest : public ::testing::Test {
74 protected: 77 protected:
75 FlexfecReceiveStreamTest() 78 FlexfecReceiveStreamTest()
76 : config_(CreateDefaultConfig(&rtcp_send_transport_)), 79 : config_(CreateDefaultConfig(&rtcp_send_transport_)),
77 receive_stream_(config_, &recovered_packet_receiver_) {} 80 receive_stream_(config_,
81 &recovered_packet_receiver_,
82 &rtt_stats_,
83 &process_thread_) {}
78 84
85 MockTransport rtcp_send_transport_;
79 FlexfecReceiveStream::Config config_; 86 FlexfecReceiveStream::Config config_;
80 MockRecoveredPacketReceiver recovered_packet_receiver_; 87 MockRecoveredPacketReceiver recovered_packet_receiver_;
81 MockTransport rtcp_send_transport_; 88 MockRtcpRttStats rtt_stats_;
89 MockProcessThread process_thread_;
82 90
83 FlexfecReceiveStreamImpl receive_stream_; 91 FlexfecReceiveStreamImpl receive_stream_;
84 }; 92 };
85 93
86 TEST_F(FlexfecReceiveStreamTest, ConstructDestruct) {} 94 TEST_F(FlexfecReceiveStreamTest, ConstructDestruct) {}
87 95
88 TEST_F(FlexfecReceiveStreamTest, StartStop) { 96 TEST_F(FlexfecReceiveStreamTest, StartStop) {
89 receive_stream_.Start(); 97 receive_stream_.Start();
90 receive_stream_.Stop(); 98 receive_stream_.Stop();
91 } 99 }
(...skipping 27 matching lines...) Expand all
119 0x00, kMediaPlType, kPayloadLength[0], kPayloadLength[1], 127 0x00, kMediaPlType, kPayloadLength[0], kPayloadLength[1],
120 kMediaTs[0], kMediaTs[1], kMediaTs[2], kMediaTs[3], 128 kMediaTs[0], kMediaTs[1], kMediaTs[2], kMediaTs[3],
121 kSsrcCount, kReservedBits, kReservedBits, kReservedBits, 129 kSsrcCount, kReservedBits, kReservedBits, kReservedBits,
122 kMediaSsrc[0], kMediaSsrc[1], kMediaSsrc[2], kMediaSsrc[3], 130 kMediaSsrc[0], kMediaSsrc[1], kMediaSsrc[2], kMediaSsrc[3],
123 kMediaSeqNum[0], kMediaSeqNum[1], kFlexfecPktMask[0], kFlexfecPktMask[1], 131 kMediaSeqNum[0], kMediaSeqNum[1], kFlexfecPktMask[0], kFlexfecPktMask[1],
124 // FEC payload. 132 // FEC payload.
125 kPayloadBits, kPayloadBits, kPayloadBits, kPayloadBits}; 133 kPayloadBits, kPayloadBits, kPayloadBits, kPayloadBits};
126 // clang-format on 134 // clang-format on
127 135
128 testing::StrictMock<MockRecoveredPacketReceiver> recovered_packet_receiver; 136 testing::StrictMock<MockRecoveredPacketReceiver> recovered_packet_receiver;
129 FlexfecReceiveStreamImpl receive_stream(config_, &recovered_packet_receiver); 137 FlexfecReceiveStreamImpl receive_stream(config_, &recovered_packet_receiver,
138 &rtt_stats_, &process_thread_);
130 139
131 // Do not call back before being started. 140 // Do not call back before being started.
132 receive_stream.AddAndProcessReceivedPacket(ParsePacket(kFlexfecPacket)); 141 receive_stream.AddAndProcessReceivedPacket(ParsePacket(kFlexfecPacket));
133 142
134 // Call back after being started. 143 // Call back after being started.
135 receive_stream.Start(); 144 receive_stream.Start();
136 EXPECT_CALL( 145 EXPECT_CALL(
137 recovered_packet_receiver, 146 recovered_packet_receiver,
138 OnRecoveredPacket(::testing::_, kRtpHeaderSize + kPayloadLength[1])); 147 OnRecoveredPacket(::testing::_, kRtpHeaderSize + kPayloadLength[1]));
139 receive_stream.AddAndProcessReceivedPacket(ParsePacket(kFlexfecPacket)); 148 receive_stream.AddAndProcessReceivedPacket(ParsePacket(kFlexfecPacket));
140 } 149 }
141 150
142 } // namespace webrtc 151 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/call/flexfec_receive_stream_impl.cc ('k') | webrtc/media/engine/webrtcvideoengine2.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698