• Kenneth Russell's avatar
    Revert "Enable SharedImages for VTVideoDecodeAccelerator" · 2f5c156b
    Kenneth Russell authored
    This reverts commit 8b7817c7.
    
    Reason for revert: assertion failures in the default configuration of webgl_conformance_tests per crbug.com/1132730 .
    
    Original change's description:
    > Enable SharedImages for VTVideoDecodeAccelerator
    >
    > Add a method to media::VideoDecodeAccelerator to indicate whether
    > SharedImage-backed picture buffers are supported. Make this return
    > true only for the VTVideoDecodeAccelerator.
    >
    > Add a parameter to media::PictureBufferManager::CreatePictureBuffers
    > to indicate if SharedImage-backed PictureBuffers are desired.
    >
    > Hook the two of these together in the ProvidePictureBuffersAsync
    > method of media::VdaVideoDecoder.
    >
    > Video playback fails with the Metal-based command decoder, because
    > it binds IOSurfaces as TEXTURE_2D, not TEXTURE_RECTANGLE. Plumb
    > through a flag to specify if TEXTURE_RECTANGLE support is present.
    >
    > Bug: 1108909
    > Change-Id: I1e42aa38e122714517724cee87eb51987645e410
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2427545
    > Commit-Queue: ccameron <ccameron@chromium.org>
    > Reviewed-by: ccameron <ccameron@chromium.org>
    > Reviewed-by: Dan Sanders <sandersd@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#811077}
    
    TBR=ccameron@chromium.org,sandersd@chromium.org,jonahr@google.com
    
    Change-Id: I21aa3be7737dc3a2d3d9d0d0cceb4fea2b1fca54
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 1108909, 1132730
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2435779Reviewed-by: default avatarKenneth Russell <kbr@chromium.org>
    Commit-Queue: Kenneth Russell <kbr@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#811306}
    2f5c156b
video_decode_accelerator.cc 5.12 KB