Commit 10ded18b authored by Ted Meyer's avatar Ted Meyer Committed by Commit Bot

Erase callback queue on NotifyError

leaving already-called-callbacks in the input_buffer_queue_ was causing
crashes when they were called a second time.



Bug: 889375
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: I6623f6736aa0551253a23282562721facf06acf0
Reviewed-on: https://chromium-review.googlesource.com/1247421Reviewed-by: default avatarFrank Liberato <liberato@chromium.org>
Commit-Queue: Ted Meyer <tmathmeyer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#594575}
parent 84fbaf84
...@@ -587,6 +587,7 @@ void D3D11VideoDecoder::NotifyError(const char* reason) { ...@@ -587,6 +587,7 @@ void D3D11VideoDecoder::NotifyError(const char* reason) {
for (auto& queue_pair : input_buffer_queue_) for (auto& queue_pair : input_buffer_queue_)
queue_pair.second.Run(DecodeStatus::DECODE_ERROR); queue_pair.second.Run(DecodeStatus::DECODE_ERROR);
input_buffer_queue_.clear();
} }
void D3D11VideoDecoder::SetCreateDeviceCallbackForTesting( void D3D11VideoDecoder::SetCreateDeviceCallbackForTesting(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment