DescriptionRevert of [Mojo Video Capture] Add test coverage for accelerated jpeg decoding (patchset #8 id:160001 of https://codereview.chromium.org/2735083002/ )
Reason for revert:
Causes a ChromiumOS test failure, see https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%281%29/builds/23888
Original issue's description:
> [Mojo Video Capture] Add test coverage for accelerated jpeg decoding
>
> This CL adds test cases to VideoCaptureBrowserTest that exercise the code path
> for accelerated jpeg decoding using a fake decode accelerator.
>
> * Add a FakeJpegDecodeAccelerator that can run in the GPU process and decodes
> jpeg frames in software using libyuv.
> * Add a command-line flag kUseFakeJpegDecodeAccelerator to enable the fake
> accelerator (and suppress any "real" ones if present).
> * Fix threading-related DCHECKs in GpuJpegDecodeAccelerator::Client to match
> what is actually happening. This is needed to make the new test cases pass.
> It also fixes issue 688184 [1].
>
> This CL is part of the Mojo Video Capture work. For the bigger picture,
> see [2] CL14.
>
> BUG=584797, 688184
> TEST=
> content_browsertests --gtest_filter="VideoCaptureBrowserTest.*"
>
> [1] https://bugs.chromium.org/p/chromium/issues/detail?id=688184
> [2] https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxsxWslJ_X-zpOhijvI8/edit?usp=sharing
> 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
>
> Review-Url: https://codereview.chromium.org/2735083002
> Cr-Commit-Position: refs/heads/master@{#458782}
> Committed: https://chromium.googlesource.com/chromium/src/+/ac2f201c499c2d65a5d43b6ce49d9af5b4520f99
TBR=posciak@chromium.org,emircan@chromium.org,mcasas@chromium.org,piman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584797, 688184
Review-Url: https://codereview.chromium.org/2766343002
Cr-Commit-Position: refs/heads/master@{#458820}
Committed: https://chromium.googlesource.com/chromium/src/+/da3f614dc0e3d04f62227cddb136997731dcd443
Patch Set 1 #Messages
Total messages: 6 (3 generated)
|