• Fergal Daly's avatar
    Revert "media/gpu/VEA unittest: Fill plane pointer and stride with NULL and 0... · f199bfbe
    Fergal Daly authored
    Revert "media/gpu/VEA unittest: Fill plane pointer and stride with NULL and 0 for 1- or 2- planes VideoFrame"
    
    This reverts commit 04f8f2c0.
    
    Reason for revert: breaks windowd compile
    
    Original change's description:
    > media/gpu/VEA unittest: Fill plane pointer and stride with NULL and 0 for 1- or 2- planes VideoFrame
    > 
    > When I enable VEA unittest to run any yuv format in crrev.com/c/1135106, I didn't
    > realize the code assumed the I420 when creating frame. Therefore, it passes
    > invalid pointer and wrong stride when the number of planes are less than 3.
    > This change fixes it by creating video frame, taking into account the number of
    > planes of input buffer format.
    > 
    > BUG=chromium:894381
    > TEST=VEA unittest for NV12 and I420 on kevin
    > 
    > Change-Id: I31e157ce9317139e7cc42a38ba77b34c13d70d70
    > Reviewed-on: https://chromium-review.googlesource.com/c/1293095
    > Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
    > Reviewed-by: Alexandre Courbot <acourbot@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#601489}
    
    TBR=hiroh@chromium.org,acourbot@chromium.org
    
    Change-Id: I02e53782dd3240da1dfd4548daefe275854601b4
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: chromium:894381
    Reviewed-on: https://chromium-review.googlesource.com/c/1293099Reviewed-by: default avatarFergal Daly <fergal@chromium.org>
    Commit-Queue: Fergal Daly <fergal@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#601491}
    f199bfbe
video_encode_accelerator_unittest.cc 96.3 KB