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

Unified Diff: media/cast/net/pacing/paced_sender.cc

Issue 1515433002: Replace uses of raw uint32's with a type-checked RtpTimeTicks data type. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Speculative workaround fix for win8_chromium_ng compile error. Created 5 years 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
« no previous file with comments | « media/cast/net/pacing/paced_sender.h ('k') | media/cast/net/pacing/paced_sender_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/cast/net/pacing/paced_sender.cc
diff --git a/media/cast/net/pacing/paced_sender.cc b/media/cast/net/pacing/paced_sender.cc
index f70fc40dab4adb58a51cabb69ee23c86b44a7b95..c751736162cf9f70f9a2765979123d6ebd32a522 100644
--- a/media/cast/net/pacing/paced_sender.cc
+++ b/media/cast/net/pacing/paced_sender.cc
@@ -335,12 +335,17 @@ void PacedSender::LogPacketEvent(const Packet& packet, CastLoggingEvent type) {
base::BigEndianReader reader(reinterpret_cast<const char*>(&packet[0]),
packet.size());
bool success = reader.Skip(4);
- success &= reader.ReadU32(&event.rtp_timestamp);
+ uint32_t truncated_rtp_timestamp;
+ success &= reader.ReadU32(&truncated_rtp_timestamp);
uint32_t ssrc;
success &= reader.ReadU32(&ssrc);
if (ssrc == audio_ssrc_) {
+ event.rtp_timestamp = last_logged_audio_rtp_timestamp_ =
+ last_logged_audio_rtp_timestamp_.Expand(truncated_rtp_timestamp);
event.media_type = AUDIO_EVENT;
} else if (ssrc == video_ssrc_) {
+ event.rtp_timestamp = last_logged_video_rtp_timestamp_ =
+ last_logged_video_rtp_timestamp_.Expand(truncated_rtp_timestamp);
event.media_type = VIDEO_EVENT;
} else {
DVLOG(3) << "Got unknown ssrc " << ssrc << " when logging packet event";
« no previous file with comments | « media/cast/net/pacing/paced_sender.h ('k') | media/cast/net/pacing/paced_sender_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698