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

Unified Diff: webrtc/video/vie_encoder.cc

Issue 2508853002: Reland of Declare VideoCodec.codec_specific_info private (Closed)
Patch Set: Rebase Created 4 years, 1 month 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 | « webrtc/config.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/video/vie_encoder.cc
diff --git a/webrtc/video/vie_encoder.cc b/webrtc/video/vie_encoder.cc
index aa93886bc38b6d14a335fb632f8fa90250d4df69..2c9c6799e8c4456e5bfd7e8ca1dfc2e4acbe5e64 100644
--- a/webrtc/video/vie_encoder.cc
+++ b/webrtc/video/vie_encoder.cc
@@ -528,19 +528,15 @@ void ViEEncoder::EncodeVideoFrame(const VideoFrame& video_frame,
webrtc::CodecSpecificInfo codec_specific_info;
codec_specific_info.codecType = webrtc::kVideoCodecVP8;
- codec_specific_info.codecSpecific.VP8.hasReceivedRPSI =
- has_received_rpsi_;
- codec_specific_info.codecSpecific.VP8.hasReceivedSLI =
- has_received_sli_;
- codec_specific_info.codecSpecific.VP8.pictureIdRPSI =
- picture_id_rpsi_;
- codec_specific_info.codecSpecific.VP8.pictureIdSLI =
- picture_id_sli_;
- has_received_sli_ = false;
- has_received_rpsi_ = false;
-
- video_sender_.AddVideoFrame(video_frame, &codec_specific_info);
- return;
+ codec_specific_info.codecSpecific.VP8.hasReceivedRPSI = has_received_rpsi_;
+ codec_specific_info.codecSpecific.VP8.hasReceivedSLI = has_received_sli_;
+ codec_specific_info.codecSpecific.VP8.pictureIdRPSI = picture_id_rpsi_;
+ codec_specific_info.codecSpecific.VP8.pictureIdSLI = picture_id_sli_;
+ has_received_sli_ = false;
+ has_received_rpsi_ = false;
+
+ video_sender_.AddVideoFrame(video_frame, &codec_specific_info);
+ return;
}
video_sender_.AddVideoFrame(video_frame, nullptr);
}
« no previous file with comments | « webrtc/config.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698