DescriptionRevert of Mac Video Capture AVFoundation: Calculate compressed frame size for MJPEG. (patchset #2 id:20001 of https://codereview.chromium.org/912293005/)
Reason for revert:
An improved version of ValidateJpeg in libyuv has been rolled into chromium and this CL is no longer needed.
Original issue's description:
> Mac Video Capture AVFoundation: Calculate compressed frame size for MJPEG.
>
> The frame size we receive from AVFoundation is sometimes excessive, e.g. the size of the uncompressed frame. For example, for a JPEG YUV422, the reported size might be 1280*720*2 = 1.8MB, while the actual compressed data is only 260kB.
>
> In these cases, libyuv::MJPGToI420 spends a lot of time in libyuv::ValidateJpeg, as much as 25%. The reason for this is that ValidateJpeg starts at the end of the buffer and loops backwards looking for the End Of Image (EOI) marker.
>
> This CL tries to optimize these cases by finding the EOI by looping forwards instead of backwards, and reporting the actual frame size to libyuv::MJPGToI420.
>
> BUG=346634, 460497
>
> Committed: https://crrev.com/dc556d2b54354adc95ee5474600f469f7ae65692
> Cr-Commit-Position: refs/heads/master@{#317333}
TBR=perkj@chromium.org,mcasas@chromium.org,tommi@chromium.org,torbjorng@google.com,torbjorng@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=346634, 460497
Committed: https://crrev.com/0deddb9627adb0a4d473c6e0fdafa8e32062101e
Cr-Commit-Position: refs/heads/master@{#325211}
Patch Set 1 #
Created: 5 years, 8 months ago
(Patch set is too large to download)
Messages
Total messages: 5 (1 generated)
|