• liberato@chromium.org's avatar
    Add null check to MediaCodecVideoDecoder::OnSurfaceDestroyed · 58013d32
    liberato@chromium.org authored
    OnSurfaceDestroyed was unconditionally dereferencing
    |target_surface_bundle_|, but this can be cleared once the codec
    enters a terminal state.  A null target bundle should just be
    ignored anyway.
    
    Bug: 806630
    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: I612c8a0c95ec0e3b918789384f44ad3a01fd2559
    Reviewed-on: https://chromium-review.googlesource.com/893638Reviewed-by: default avatarThomas Guilbert <tguilbert@chromium.org>
    Commit-Queue: Frank Liberato <liberato@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#532974}
    58013d32
media_codec_video_decoder.cc 29.3 KB