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

Side by Side Diff: webrtc/modules/rtp_rtcp/include/rtp_rtcp.h

Issue 1947913002: Move, almost, all receive side references to RTP to RtpStreamReceiver. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Addressed comments in PS#4 Created 4 years, 7 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 | « no previous file | webrtc/video/rtp_stream_receiver.h » ('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) 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 522 matching lines...) Expand 10 before | Expand all | Expand 10 after
533 * 533 *
534 * return -1 on failure else 0 534 * return -1 on failure else 0
535 */ 535 */
536 // TODO(philipel): Deprecate this and start using SendNack instead, 536 // TODO(philipel): Deprecate this and start using SendNack instead,
537 // mostly because we want a function that actually send 537 // mostly because we want a function that actually send
538 // NACK for the specified packets. 538 // NACK for the specified packets.
539 virtual int32_t SendNACK(const uint16_t* nackList, uint16_t size) = 0; 539 virtual int32_t SendNACK(const uint16_t* nackList, uint16_t size) = 0;
540 540
541 /* 541 /*
542 * Send NACK for the packets specified. 542 * Send NACK for the packets specified.
543 *
544 * Note: This assumes the caller keeps track of timing and doesn't rely on
545 * the RTP module to do this.
543 */ 546 */
544 virtual void SendNack(const std::vector<uint16_t>& sequence_numbers) = 0; 547 virtual void SendNack(const std::vector<uint16_t>& sequence_numbers) = 0;
545 548
546 /* 549 /*
547 * Store the sent packets, needed to answer to a Negative acknowledgement 550 * Store the sent packets, needed to answer to a Negative acknowledgement
548 * requests 551 * requests
549 */ 552 */
550 virtual void SetStorePacketsStatus(bool enable, uint16_t numberToStore) = 0; 553 virtual void SetStorePacketsStatus(bool enable, uint16_t numberToStore) = 0;
551 554
552 // Returns true if the module is configured to store packets. 555 // Returns true if the module is configured to store packets.
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
644 647
645 /* 648 /*
646 * send a request for a keyframe 649 * send a request for a keyframe
647 * 650 *
648 * return -1 on failure else 0 651 * return -1 on failure else 0
649 */ 652 */
650 virtual int32_t RequestKeyFrame() = 0; 653 virtual int32_t RequestKeyFrame() = 0;
651 }; 654 };
652 } // namespace webrtc 655 } // namespace webrtc
653 #endif // WEBRTC_MODULES_RTP_RTCP_INCLUDE_RTP_RTCP_H_ 656 #endif // WEBRTC_MODULES_RTP_RTCP_INCLUDE_RTP_RTCP_H_
OLDNEW
« no previous file with comments | « no previous file | webrtc/video/rtp_stream_receiver.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698