• Xiaohan Wang's avatar
    media: Handle GetCdm() failure in AndroidVideoDecodeAccelerator · e636cd89
    Xiaohan Wang authored
    In normal cases in the render process WebMediaPlayerImpl holds a
    reference to the CDM such that the CDM should never be destructed before
    the media pipeline. Howeven, when both the decoder and CDM are hosted
    remotely (e.g. in the GPU process on Android), the destruction order
    could be reversed due to IPC. In this case, we should just fail instead
    of crash.
    
    Note that this case is already handled in MojoAudioDecoderService,
    MojoVideoDecoderService and MojoRendererService.
    
    Bug: 833641
    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;master.tryserver.chromium.win:win_optional_gpu_tests_rel
    Change-Id: I7f07c07dea86033ebe57700e0ce8834ea94b06f3
    Reviewed-on: https://chromium-review.googlesource.com/1013690Reviewed-by: default avatarFrank Liberato <liberato@chromium.org>
    Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#551211}
    e636cd89
android_video_decode_accelerator.cc 68.5 KB