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

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

Issue 196433002: Cast: Log sender side packet events. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 years, 9 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 e23b952a97d3024a926a37a7f0e058701c91e7fa..c31b4d97882f26ff14f4b6cf39013fd8d2ed3a05 100644
--- a/media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer.h
+++ b/media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer.h
@@ -12,8 +12,16 @@
#include "base/time/time.h"
#include "media/cast/transport/rtp_sender/packet_storage/packet_storage.h"
+namespace base {
+
+class TickClock;
+}
+
namespace media {
namespace cast {
+
+class LoggingImpl;
+
namespace transport {
class PacedSender;
@@ -47,7 +55,9 @@ class RtpPacketizer {
public:
RtpPacketizer(PacedSender* const transport,
PacketStorage* packet_storage,
- RtpPacketizerConfig rtp_packetizer_config);
+ RtpPacketizerConfig rtp_packetizer_config,
+ base::TickClock* clock,
+ LoggingImpl* logging);
~RtpPacketizer();
// The video_frame objects ownership is handled by the main cast thread.
@@ -81,6 +91,8 @@ class RtpPacketizer {
RtpPacketizerConfig config_;
PacedSender* const transport_; // Not owned by this class.
PacketStorage* packet_storage_;
+ base::TickClock* const clock_; // Not owned by this class.
+ LoggingImpl* const logging_; // Not owned by this class.
base::TimeTicks time_last_sent_rtp_timestamp_;
uint16 sequence_number_;

Powered by Google App Engine
This is Rietveld 408576698