The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1e1ea2bcd2e632ba3b3dcbd45ca3405771e57a9c

commit 1e1ea2bcd2e632ba3b3dcbd45ca3405771e57a9c
Author: watk <watk@chromium.org>
Date: Wed Feb 15 05:41:30 2017

media: Don't create a new MediaCodec during AVDA teardown

OnDrainCompleted() is always called during AVDA teardown, and it
previously always called ResetCodecState() which resulted in
unnecessarily creating new MediaCodecs during shutdown whenever state_
was ERROR, or on older devices where flush() doesn't work.

Now we don't bother calling ResetCodecState() during destruction, and
we also don't try to recover from errors. Errors are generally
unrecoverable, so retrying is usually a bad idea.

BUG= 688030 
CQ_INCLUDE_TRYBOTS=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

Review-Url: https://codereview.chromium.org/2662173002
Cr-Commit-Position: refs/heads/master@{#450565}

[modify] https://crrev.com/1e1ea2bcd2e632ba3b3dcbd45ca3405771e57a9c/media/gpu/android_video_decode_accelerator.cc
[modify] https://crrev.com/1e1ea2bcd2e632ba3b3dcbd45ca3405771e57a9c/media/gpu/android_video_decode_accelerator.h
[modify] https://crrev.com/1e1ea2bcd2e632ba3b3dcbd45ca3405771e57a9c/media/gpu/android_video_decode_accelerator_unittest.cc
[modify] https://crrev.com/1e1ea2bcd2e632ba3b3dcbd45ca3405771e57a9c/media/gpu/avda_codec_allocator.h
[modify] https://crrev.com/1e1ea2bcd2e632ba3b3dcbd45ca3405771e57a9c/media/gpu/gpu_video_decode_accelerator_factory.cc