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

Unified Diff: media/cast/video_sender/video_sender.cc

Issue 288103002: [Cast] EncodedAudioFrame+EncodedVideoFrame+reference_time --> EncodedFrame (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase 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
« no previous file with comments | « media/cast/video_sender/video_sender.h ('k') | media/cast/video_sender/video_sender_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/cast/video_sender/video_sender.cc
diff --git a/media/cast/video_sender/video_sender.cc b/media/cast/video_sender/video_sender.cc
index 5a1546359097f4447d1d251b005b1d21afd7e5f7..bd2c0138babb1b0c693a9780215fdc5316cc4baa 100644
--- a/media/cast/video_sender/video_sender.cc
+++ b/media/cast/video_sender/video_sender.cc
@@ -147,7 +147,7 @@ void VideoSender::InsertRawVideoFrame(
"cast_perf_test", "InsertRawVideoFrame",
TRACE_EVENT_SCOPE_THREAD,
"timestamp", capture_time.ToInternalValue(),
- "rtp_timestamp", GetVideoRtpTimestamp(capture_time));
+ "rtp_timestamp", rtp_timestamp);
if (video_encoder_->EncodeVideoFrame(
video_frame,
@@ -162,14 +162,11 @@ void VideoSender::InsertRawVideoFrame(
void VideoSender::SendEncodedVideoFrameMainThread(
int requested_bitrate_before_encode,
- scoped_ptr<transport::EncodedVideoFrame> encoded_frame,
- const base::TimeTicks& capture_time) {
+ scoped_ptr<transport::EncodedFrame> encoded_frame) {
DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::MAIN));
last_send_time_ = cast_environment_->Clock()->NowTicks();
- if (encoded_frame->key_frame) {
- VLOG(1) << "Send encoded key frame; frame_id:"
- << static_cast<int>(encoded_frame->frame_id);
- }
+ VLOG_IF(1, encoded_frame->dependency == transport::EncodedFrame::KEY)
+ << "Send encoded key frame; frame_id: " << encoded_frame->frame_id;
DCHECK_GT(frames_in_encoder_, 0);
frames_in_encoder_--;
@@ -177,22 +174,23 @@ void VideoSender::SendEncodedVideoFrameMainThread(
cast_environment_->Logging()->InsertEncodedFrameEvent(
last_send_time_, FRAME_ENCODED, VIDEO_EVENT, encoded_frame->rtp_timestamp,
frame_id, static_cast<int>(encoded_frame->data.size()),
- encoded_frame->key_frame,
+ encoded_frame->dependency == transport::EncodedFrame::KEY,
requested_bitrate_before_encode);
// Used by chrome/browser/extension/api/cast_streaming/performance_test.cc
TRACE_EVENT_INSTANT1(
"cast_perf_test", "VideoFrameEncoded",
TRACE_EVENT_SCOPE_THREAD,
- "rtp_timestamp", GetVideoRtpTimestamp(capture_time));
+ "rtp_timestamp", encoded_frame->rtp_timestamp);
// Only use lowest 8 bits as key.
frame_id_to_rtp_timestamp_[frame_id & 0xff] = encoded_frame->rtp_timestamp;
last_sent_frame_id_ = static_cast<int>(encoded_frame->frame_id);
- rtp_timestamp_helper_.StoreLatestTime(capture_time,
+ DCHECK(!encoded_frame->reference_time.is_null());
+ rtp_timestamp_helper_.StoreLatestTime(encoded_frame->reference_time,
encoded_frame->rtp_timestamp);
- transport_sender_->InsertCodedVideoFrame(encoded_frame.get(), capture_time);
+ transport_sender_->InsertCodedVideoFrame(*encoded_frame);
UpdateFramesInFlight();
InitializeTimers();
}
« no previous file with comments | « media/cast/video_sender/video_sender.h ('k') | media/cast/video_sender/video_sender_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698