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

Unified Diff: media/cast/receiver/cast_receiver_impl.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/receiver/cast_receiver_impl.h ('k') | media/cast/receiver/frame_receiver.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/cast/receiver/cast_receiver_impl.cc
diff --git a/media/cast/receiver/cast_receiver_impl.cc b/media/cast/receiver/cast_receiver_impl.cc
index 076bd26559110ba1640669fdc844396f82500df6..0c7dcedc83ac37e78bd8c8fd5a6ebe3b64b52ea1 100644
--- a/media/cast/receiver/cast_receiver_impl.cc
+++ b/media/cast/receiver/cast_receiver_impl.cc
@@ -127,7 +127,7 @@ void CastReceiverImpl::DecodeEncodedAudioFrame(
audio_codec_));
}
const uint32_t frame_id = encoded_frame->frame_id;
- const uint32_t rtp_timestamp = encoded_frame->rtp_timestamp;
+ const RtpTimeTicks rtp_timestamp = encoded_frame->rtp_timestamp;
const base::TimeTicks playout_time = encoded_frame->reference_time;
audio_decoder_->DecodeFrame(
std::move(encoded_frame),
@@ -149,13 +149,13 @@ void CastReceiverImpl::DecodeEncodedVideoFrame(
TRACE_EVENT_INSTANT2(
"cast_perf_test", "PullEncodedVideoFrame",
TRACE_EVENT_SCOPE_THREAD,
- "rtp_timestamp", encoded_frame->rtp_timestamp,
+ "rtp_timestamp", encoded_frame->rtp_timestamp.lower_32_bits(),
"render_time", encoded_frame->reference_time.ToInternalValue());
if (!video_decoder_)
video_decoder_.reset(new VideoDecoder(cast_environment_, video_codec_));
const uint32_t frame_id = encoded_frame->frame_id;
- const uint32_t rtp_timestamp = encoded_frame->rtp_timestamp;
+ const RtpTimeTicks rtp_timestamp = encoded_frame->rtp_timestamp;
const base::TimeTicks playout_time = encoded_frame->reference_time;
video_decoder_->DecodeFrame(
std::move(encoded_frame),
@@ -168,7 +168,7 @@ void CastReceiverImpl::EmitDecodedAudioFrame(
const scoped_refptr<CastEnvironment>& cast_environment,
const AudioFrameDecodedCallback& callback,
uint32_t frame_id,
- uint32_t rtp_timestamp,
+ RtpTimeTicks rtp_timestamp,
const base::TimeTicks& playout_time,
scoped_ptr<AudioBus> audio_bus,
bool is_continuous) {
@@ -195,7 +195,7 @@ void CastReceiverImpl::EmitDecodedVideoFrame(
const scoped_refptr<CastEnvironment>& cast_environment,
const VideoFrameDecodedCallback& callback,
uint32_t frame_id,
- uint32_t rtp_timestamp,
+ RtpTimeTicks rtp_timestamp,
const base::TimeTicks& playout_time,
const scoped_refptr<VideoFrame>& video_frame,
bool is_continuous) {
@@ -217,7 +217,7 @@ void CastReceiverImpl::EmitDecodedVideoFrame(
TRACE_EVENT_INSTANT1(
"cast_perf_test", "FrameDecoded",
TRACE_EVENT_SCOPE_THREAD,
- "rtp_timestamp", rtp_timestamp);
+ "rtp_timestamp", rtp_timestamp.lower_32_bits());
}
callback.Run(video_frame, playout_time, is_continuous);
« no previous file with comments | « media/cast/receiver/cast_receiver_impl.h ('k') | media/cast/receiver/frame_receiver.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698