• Miguel Casas's avatar
    vaapi vda: Delete owned objects on worker thread in Cleanup() · 70340ce0
    Miguel Casas authored
    This CL adds a SEQUENCE_CHECKER to Vaapi*Accelerator classes, and
    posts the destruction of those objects to the appropriate thread on
    Cleanup().
    
    Also makes {H264,VP8,VP9}Picture RefCountedThreadSafe, see miu@
    comment in
    https://chromium-review.googlesource.com/c/chromium/src/+/794091#message-a64bed985cfaf8a19499a517bb110a7ce581dc0f
    
    TEST=play back VP9/VP8/H264 w/ simplechrome on soraka, Release build
    unstripped, let video play for a few seconds then navigate back; no
    crashes. Unittests as before:
    video_decode_accelerator_unittest --test_video_data=test-25fps.vp9:320:240:250:250:35:150:12
    video_decode_accelerator_unittest --test_video_data=test-25fps.vp8:320:240:250:250:35:150:11
    video_decode_accelerator_unittest --test_video_data=test-25fps.h264:320:240:250:258:35:150:1
    
    Bug: 789160
    Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
    Change-Id: I7d96aaf89c92bf46f00c8b8b36798e057a842ed2
    Reviewed-on: https://chromium-review.googlesource.com/794091Reviewed-by: default avatarPawel Osciak <posciak@chromium.org>
    Commit-Queue: Miguel Casas <mcasas@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#523372}
    70340ce0
vp9_picture.h 1.05 KB