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

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

Issue 178133005: Audit/fix use of media::VideoFrame::coded_size() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: e6f9affb danakj@ comments. Created 6 years, 10 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/video_sender/external_video_encoder.cc
diff --git a/media/cast/video_sender/external_video_encoder.cc b/media/cast/video_sender/external_video_encoder.cc
index 6ec8d3df57705a744c94dd45be750aceb14a82de..69f550a2770bade437dac7381c7ff37e6f145869 100644
--- a/media/cast/video_sender/external_video_encoder.cc
+++ b/media/cast/video_sender/external_video_encoder.cc
@@ -156,15 +156,15 @@ class LocalVideoEncodeAcceleratorClient
// Do a stride copy of the input frame to match the input requirements.
media::CopyYPlane(video_frame->data(VideoFrame::kYPlane),
video_frame->stride(VideoFrame::kYPlane),
- video_frame->natural_size().height(),
+ video_frame->coded_size().height(),
frame.get());
media::CopyUPlane(video_frame->data(VideoFrame::kUPlane),
video_frame->stride(VideoFrame::kUPlane),
- video_frame->natural_size().height(),
+ video_frame->coded_size().height(),
frame.get());
media::CopyVPlane(video_frame->data(VideoFrame::kVPlane),
video_frame->stride(VideoFrame::kVPlane),
- video_frame->natural_size().height(),
+ video_frame->coded_size().height(),
frame.get());
encoded_frame_data_storage_.push_back(

Powered by Google App Engine
This is Rietveld 408576698