GitLab

The destructor of class VideoCaptureJpegDecoderImpl was blocking the
current thread, which could lead to a deadlock described in
https://bugs.chromium.org/p/chromium/issues/detail?id=921694#c6.
This deadlock has shown up as test flakiness.

This CL removes the need for blocking the thread in the destructor
by requiring that instances are always destroyed on the thread we
would otherwise have to wait for.

This CL also reenables test cases that were disabled because of the
flakiness.

Test: content_browsertests --gtest_filter=*ReceiveFramesFromFakeCaptureDevice*
Bug: 921694
Change-Id: I06b960c8e9cf749b94ce7785a3726884fd47a37d
Reviewed-on: https://chromium-review.googlesource.com/c/1423638
Commit-Queue: Christian Fremerey <chfremer@chromium.org>
Reviewed-by: Emircan Uysaler <emircan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#624957}