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

Unified Diff: media/cast/transport/transport_video_sender.cc

Issue 288103002: [Cast] EncodedAudioFrame+EncodedVideoFrame+reference_time --> EncodedFrame (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed hclam@'s first round comments. Fixed chrome unit_tests compiles. 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
Index: media/cast/transport/transport_video_sender.cc
diff --git a/media/cast/transport/transport_video_sender.cc b/media/cast/transport/transport_video_sender.cc
index ef120bcb0547802e88a91ab28bb1224669e9d4a4..63cf8ced5e7837920b0d5173e5105db7f4986d45 100644
--- a/media/cast/transport/transport_video_sender.cc
+++ b/media/cast/transport/transport_video_sender.cc
@@ -27,31 +27,24 @@ TransportVideoSender::TransportVideoSender(
TransportVideoSender::~TransportVideoSender() {}
-void TransportVideoSender::InsertCodedVideoFrame(
- const EncodedVideoFrame* coded_frame,
- const base::TimeTicks& capture_time) {
+void TransportVideoSender::SendFrame(const EncodedFrame& video_frame) {
if (!initialized_) {
return;
}
if (encryptor_.initialized()) {
- EncodedVideoFrame encrypted_video_frame;
-
- if (!EncryptVideoFrame(*coded_frame, &encrypted_video_frame))
+ EncodedFrame encrypted_frame;
+ if (!EncryptVideoFrame(video_frame, &encrypted_frame)) {
+ NOTREACHED();
return;
-
- rtp_sender_.IncomingEncodedVideoFrame(&encrypted_video_frame, capture_time);
+ }
+ rtp_sender_.SendFrame(encrypted_frame);
} else {
- rtp_sender_.IncomingEncodedVideoFrame(coded_frame, capture_time);
- }
- if (coded_frame->key_frame) {
- VLOG(1) << "Send encoded key frame; frame_id:"
- << static_cast<int>(coded_frame->frame_id);
+ rtp_sender_.SendFrame(video_frame);
}
}
bool TransportVideoSender::EncryptVideoFrame(
- const EncodedVideoFrame& video_frame,
- EncodedVideoFrame* encrypted_frame) {
+ const EncodedFrame& video_frame, EncodedFrame* encrypted_frame) {
if (!initialized_) {
return false;
}
@@ -59,11 +52,11 @@ bool TransportVideoSender::EncryptVideoFrame(
video_frame.frame_id, video_frame.data, &(encrypted_frame->data)))
return false;
- encrypted_frame->codec = video_frame.codec;
- encrypted_frame->key_frame = video_frame.key_frame;
+ encrypted_frame->relationship = video_frame.relationship;
encrypted_frame->frame_id = video_frame.frame_id;
- encrypted_frame->last_referenced_frame_id =
- video_frame.last_referenced_frame_id;
+ encrypted_frame->referenced_frame_id = video_frame.referenced_frame_id;
+ encrypted_frame->rtp_timestamp = video_frame.rtp_timestamp;
+ encrypted_frame->reference_time = video_frame.reference_time;
return true;
}

Powered by Google App Engine
This is Rietveld 408576698