let me take a look

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/801b0340f44d7c38e6d97d42fedb6ff79f2994ea

commit 801b0340f44d7c38e6d97d42fedb6ff79f2994ea
Author: Vadim Bendebury <vbendeb@chromium.org>
Date: Tue May 03 20:04:44 2016

autotest-chrome: exclude video tests on veyron platforms

This is a temp patch until the root cause of the issue is found and fixed.

BUG= chromium:608838 
TEST=verified emerge works

Change-Id: Ia7f30fcb80f2c132ae0ac5ad2d27404bd7862eeb
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/342080
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/801b0340f44d7c38e6d97d42fedb6ff79f2994ea/chromeos-base/autotest-chrome/autotest-chrome-9999.ebuild

What's an eta for the revert of this CL? These are very important tests and we don't want to lose video test coverage from these tests.

Is there a better way to handle platform specific regressions so the whole tree doesn't get affected?

just in case this is not clear: these tests have been disabled only for veyron platforms. Can this wait til posciak@ is back in the office (apparently next week)?

If not - I am not sure who else is the right person to look into this, marcheu@ would know?

Yes posciak and tfiga.

and one more thing, the patch description is misleading, the only excluded tests are

video_ChromeHWDecodeUsed
video_ChromeRTCHWDecodeUsed
video_ChromeRTCHWEncodeUsed
video_ChromeVidResChangeHWDecode

+tfiga in case he can help debug sooner than next week.

kcwu can you look into this?

posciak messaged me and suggested wuchengli@ might be able to help.

I will take a look.

My CL to fix this issue: https://codereview.chromium.org/1946153002/

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

commit 152f266aafe81cf9ef79aec4b57d9e80cea4fef0
Author: kcwu <kcwu@chromium.org>
Date: Thu May 05 21:38:38 2016

Fix V4L2 device failed to create on veyron board

This was broken since CL 6e70beb8, gpu & media refactoring.

This CL modified the build rule, so dependencies of media/gpu will define USE_LIBV4L2 automatically, which is needed by
content/common/sandbox_linux/bpf_gpu_policy_linux.cc.

TEST=play video on veyron and VDA works
BUG=608809, 608838 , 586386 

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

[modify] https://crrev.com/152f266aafe81cf9ef79aec4b57d9e80cea4fef0/media/gpu/BUILD.gn
[modify] https://crrev.com/152f266aafe81cf9ef79aec4b57d9e80cea4fef0/media/gpu/ipc/client/BUILD.gn
[modify] https://crrev.com/152f266aafe81cf9ef79aec4b57d9e80cea4fef0/media/media_gpu.gypi

Can the auto test be re enabled?

This was fixed in 8289.0.0 / 52.0.2726.0. Kuang-che. Please re-enable the test.

https://chromium-review.googlesource.com/#/c/342080/ needs to be reverted to re-enable the test.

CL to re-enable test https://chromium-review.googlesource.com/#/c/343433/

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/c252b61a4a2f371340345b5b8c4e65d556c42d58

commit c252b61a4a2f371340345b5b8c4e65d556c42d58
Author: Kuang-che Wu <kcwu@chromium.org>
Date: Mon May 09 09:25:59 2016

Revert "autotest-chrome: exclude video tests on veyron platforms"

This reverts commit 801b0340f44d7c38e6d97d42fedb6ff79f2994ea.

Veyron video HW acceleration is fixed.

BUG= chromium:608838 
TEST=verified emerge works

Change-Id: I2532778311e81efd79095fae2591241f4a0c7380
Reviewed-on: https://chromium-review.googlesource.com/343433
Commit-Ready: Kuang-che Wu <kcwu@chromium.org>
Tested-by: Kuang-che Wu <kcwu@chromium.org>
Reviewed-by: Wu-cheng Li <wuchengli@chromium.org>

[modify] https://crrev.com/c252b61a4a2f371340345b5b8c4e65d556c42d58/chromeos-base/autotest-chrome/autotest-chrome-9999.ebuild

Closing as the tests are now reenabled.

Bulk verified

bulk verified