From: Dan Minor Date: Thu, 31 Jan 2019 15:37:00 -0500 Subject: Bug 1524208 - Calculate stride based upon target_width in video_capture_impl.cc; r=pehrsons Differential Revision: https://phabricator.services.mozilla.com/D18270 Mercurial Revision: https://hg.mozilla.org/mozilla-central/rev/51d12094d825c4c4467cb132d03d4f3cad4b1b82 --- modules/video_capture/video_capture_impl.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/modules/video_capture/video_capture_impl.cc b/modules/video_capture/video_capture_impl.cc index 7071776d0a..02404697ad 100644 --- a/modules/video_capture/video_capture_impl.cc +++ b/modules/video_capture/video_capture_impl.cc @@ -180,8 +180,6 @@ int32_t VideoCaptureImpl::IncomingFrame(uint8_t* videoFrame, } } - int stride_y = width; - int stride_uv = (width + 1) / 2; int target_width = width; int target_height = abs(height); @@ -194,6 +192,9 @@ int32_t VideoCaptureImpl::IncomingFrame(uint8_t* videoFrame, } } + int stride_y = target_width; + int stride_uv = (target_width + 1) / 2; + // Setting absolute height (in case it was negative). // In Windows, the image starts bottom left, instead of top left. // Setting a negative source height, inverts the image (within LibYuv).