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

Unified Diff: media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer.h

Issue 281453003: Cast: Simplify code path for RTCP sender report (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merged Created 6 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 side-by-side diff with in-line comments
Download patch
Index: media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer.h
diff --git a/media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer.h b/media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer.h
index 2b90638430895b944cf8007c1064a06a32e6a62c..ff5ad84f577bb58201c05208b8a2d71b70117d84 100644
--- a/media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer.h
+++ b/media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer.h
@@ -56,23 +56,21 @@ class RtpPacketizer {
~RtpPacketizer();
// The video_frame objects ownership is handled by the main cast thread.
+ // |capture_time| is only used for scheduling of outgoing packets.
void IncomingEncodedVideoFrame(const EncodedVideoFrame* video_frame,
const base::TimeTicks& capture_time);
// The audio_frame objects ownership is handled by the main cast thread.
+ // |recorded_time| is only used for scheduling of outgoing packets.
void IncomingEncodedAudioFrame(const EncodedAudioFrame* audio_frame,
const base::TimeTicks& recorded_time);
- bool LastSentTimestamp(base::TimeTicks* time_sent,
- uint32* rtp_timestamp) const;
-
// Return the next sequence number, and increment by one. Enables unique
// incremental sequence numbers for every packet (including retransmissions).
uint16 NextSequenceNumber();
- int send_packets_count() { return send_packets_count_; }
-
- size_t send_octet_count() { return send_octet_count_; }
+ size_t send_packet_count() const { return send_packet_count_; }
+ size_t send_octet_count() const { return send_octet_count_; }
private:
void Cast(bool is_key,
@@ -88,12 +86,11 @@ class RtpPacketizer {
PacedSender* const transport_; // Not owned by this class.
PacketStorage* packet_storage_;
- base::TimeTicks time_last_sent_rtp_timestamp_;
uint16 sequence_number_;
uint32 rtp_timestamp_;
uint16 packet_id_;
- int send_packets_count_;
+ size_t send_packet_count_;
size_t send_octet_count_;
};
« no previous file with comments | « media/cast/transport/cast_transport_sender_impl.cc ('k') | media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698