• Miguel Casas's avatar
    VaapiPictureFactory: correct DCHECK · bd4386f5
    Miguel Casas authored
    crrev.com/c/1100535 landed a DCHECK that does not verify from ARC++,
    
    
    [1814:1814:0619/160631.868318:FATAL:vaapi_picture_factory.cc(48)] Check failed: picture_buffer.texture_target() == GetGLTextureTarget() (0 vs. 36197)
    #0 0x59c50521b4dc base::debug::StackTrace::StackTrace()
    #1 0x59c50518f620 logging::LogMessage::~LogMessage()
    #2 0x59c5026cc721 media::VaapiPictureFactory::Create()
    #3 0x59c5026d2a94 media::VaapiVideoDecodeAccelerator::AssignPictureBuffers()
    #4 0x59c506967623 arc::GpuArcVideoDecodeAccelerator::AssignPictureBuffers()
    #5 0x59c50269c433 arc::mojom::VideoDecodeAcceleratorStubDispatch::Accept()
    ...
    
    This CL corrects it.
    
    Bug: 822346
    Test: YT App playing back correctly with dcheck_always_on=true on nautilus.
    Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
    Change-Id: Icff42847518df24dcc38998acdc2903e7928469f
    Reviewed-on: https://chromium-review.googlesource.com/1106670Reviewed-by: default avatarHirokazu Honda <hiroh@chromium.org>
    Commit-Queue: Miguel Casas <mcasas@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#568703}
    bd4386f5
vaapi_picture_factory.cc 4.02 KB