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

Side by Side Diff: webrtc/video/rtp_stream_receiver.h

Issue 2826263004: Move responsibility for RTP header extensions on video receive. (Closed)
Patch Set: Created 3 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 132 matching lines...) Expand 10 before | Expand all | Expand 10 after
143 bool in_order); 143 bool in_order);
144 // Parses and handles for instance RTX and RED headers. 144 // Parses and handles for instance RTX and RED headers.
145 // This function assumes that it's being called from only one thread. 145 // This function assumes that it's being called from only one thread.
146 bool ParseAndHandleEncapsulatingHeader(const uint8_t* packet, 146 bool ParseAndHandleEncapsulatingHeader(const uint8_t* packet,
147 size_t packet_length, 147 size_t packet_length,
148 const RTPHeader& header); 148 const RTPHeader& header);
149 void NotifyReceiverOfFecPacket(const RTPHeader& header); 149 void NotifyReceiverOfFecPacket(const RTPHeader& header);
150 bool IsPacketInOrder(const RTPHeader& header) const; 150 bool IsPacketInOrder(const RTPHeader& header) const;
151 bool IsPacketRetransmitted(const RTPHeader& header, bool in_order) const; 151 bool IsPacketRetransmitted(const RTPHeader& header, bool in_order) const;
152 void UpdateHistograms(); 152 void UpdateHistograms();
153 #if 0
153 void EnableReceiveRtpHeaderExtension(const std::string& extension, int id); 154 void EnableReceiveRtpHeaderExtension(const std::string& extension, int id);
155 #endif
154 bool IsRedEnabled() const; 156 bool IsRedEnabled() const;
155 void InsertSpsPpsIntoTracker(uint8_t payload_type); 157 void InsertSpsPpsIntoTracker(uint8_t payload_type);
156 158
157 Clock* const clock_; 159 Clock* const clock_;
158 // Ownership of this object lies with VideoReceiveStream, which owns |this|. 160 // Ownership of this object lies with VideoReceiveStream, which owns |this|.
159 const VideoReceiveStream::Config& config_; 161 const VideoReceiveStream::Config& config_;
160 PacketRouter* const packet_router_; 162 PacketRouter* const packet_router_;
161 ProcessThread* const process_thread_; 163 ProcessThread* const process_thread_;
162 164
163 RemoteNtpTimeEstimator ntp_estimator_; 165 RemoteNtpTimeEstimator ntp_estimator_;
(...skipping 26 matching lines...) Expand all
190 // TODO(johan): Remove pt_codec_params_ once 192 // TODO(johan): Remove pt_codec_params_ once
191 // https://bugs.chromium.org/p/webrtc/issues/detail?id=6883 is resolved. 193 // https://bugs.chromium.org/p/webrtc/issues/detail?id=6883 is resolved.
192 // Maps a payload type to a map of out-of-band supplied codec parameters. 194 // Maps a payload type to a map of out-of-band supplied codec parameters.
193 std::map<uint8_t, std::map<std::string, std::string>> pt_codec_params_; 195 std::map<uint8_t, std::map<std::string, std::string>> pt_codec_params_;
194 int16_t last_payload_type_ = -1; 196 int16_t last_payload_type_ = -1;
195 }; 197 };
196 198
197 } // namespace webrtc 199 } // namespace webrtc
198 200
199 #endif // WEBRTC_VIDEO_RTP_STREAM_RECEIVER_H_ 201 #endif // WEBRTC_VIDEO_RTP_STREAM_RECEIVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698