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

Unified Diff: media/cast/sender/vp8_encoder.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/sender/video_sender.cc ('k') | media/cast/test/cast_benchmarks.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/cast/sender/vp8_encoder.cc
diff --git a/media/cast/sender/vp8_encoder.cc b/media/cast/sender/vp8_encoder.cc
index bcb137826f4785f302c013136009e7b28744ce36..f2d6117f657f6b34cd5402c9f6f93d7bbd1eee55 100644
--- a/media/cast/sender/vp8_encoder.cc
+++ b/media/cast/sender/vp8_encoder.cc
@@ -226,7 +226,7 @@ void Vp8Encoder::Encode(const scoped_refptr<media::VideoFrame>& video_frame,
encoded_frame->referenced_frame_id = last_encoded_frame_id_ - 1;
}
encoded_frame->rtp_timestamp =
- TimeDeltaToRtpDelta(video_frame->timestamp(), kVideoFrequency);
+ RtpTimeTicks::FromTimeDelta(video_frame->timestamp(), kVideoFrequency);
encoded_frame->reference_time = reference_time;
encoded_frame->data.assign(
static_cast<const uint8_t*>(pkt->data.frame.buf),
@@ -245,7 +245,7 @@ void Vp8Encoder::Encode(const scoped_refptr<media::VideoFrame>& video_frame,
const std::string details = base::StringPrintf(
"SV/%c,id=%" PRIu32 ",rtp=%" PRIu32 ",br=%d,kfr=%c",
encoded_frame->dependency == EncodedFrame::KEY ? 'K' : 'D',
- encoded_frame->frame_id, encoded_frame->rtp_timestamp,
+ encoded_frame->frame_id, encoded_frame->rtp_timestamp.lower_32_bits(),
static_cast<int>(config_.rc_target_bitrate),
key_frame_requested_ ? 'Y' : 'N');
base::debug::SetCrashKeyValue(kZeroEncodeDetails, details);
« no previous file with comments | « media/cast/sender/video_sender.cc ('k') | media/cast/test/cast_benchmarks.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698