New issue
Advanced search Search tips

Issue 665324 link

Starred by 1 user
Status: Fixed
Owner:
jcliang@chromium.org
Closed: Nov 2016
Cc:
rohi...@chromium.org
vsu...@chromium.org
posciak@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Remove hardware_VideoDecodeCapable test

Project Member Reported by wuchengli@chromium.org, Nov 15 2016 
We plan to replace labels with media_codecs.xml (http://crbug.com/663285). So hardware_VideoDecodeCapable won't be needed.

hardware_VideoDecodeCapable is not running on any device because no device has vaapi or video_decode_v4l2 labels. I looked at the checked formats in the test and it's not up-to-date for all devices. Also, Nvidia tried to run the test and it failed because nyan doesn't use v4l2 devices. We should remove the test to avoid confusion.

Comment 1 by jcliang@chromium.org, Nov 15 2016

Blockedon: 663285
I guess this one is blocked on issue 663285.

Comment 2 by wuchengli@chromium.org, Nov 15 2016

Blockedon: -663285
It's fine to remove hardware_VideoDecodeCapable now. hardware_VideoDecodeCapable is not maintined and not running anyway.

Comment 3 by jcliang@chromium.org, Nov 21 2016

Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 25 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/fe016fc895f98c076ebb4312fc877c018e98e2cd

commit fe016fc895f98c076ebb4312fc877c018e98e2cd
Author: Ricky Liang <jcliang@chromium.org>
Date: Fri Nov 18 05:54:09 2016

Remove hardware_VideoDecodeCapable test

This test is obsolete. Let's remove it to avoid confusion.

BUG= chromium:665324 
CQ-DEPEND=CL:413073
TEST=cbuildbot --remote -g '412306' daisy-release elm-release samus-release

Change-Id: Ic209934b9f1aeb3afbd9a8d7a6e1b0db9827ad96
Reviewed-on: https://chromium-review.googlesource.com/412306
Commit-Ready: Ricky Liang <jcliang@chromium.org>
Tested-by: Ricky Liang <jcliang@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>
Reviewed-by: Wu-cheng Li <wuchengli@chromium.org>

[delete] https://crrev.com/2cfc851f8b6ba6e899418774be8d761b37efe700/client/site_tests/hardware_VideoDecodeCapable/src/setup_v4l2.py
[delete] https://crrev.com/2cfc851f8b6ba6e899418774be8d761b37efe700/client/site_tests/hardware_VideoDecodeCapable/control.v4l2
[delete] https://crrev.com/2cfc851f8b6ba6e899418774be8d761b37efe700/client/site_tests/hardware_VideoDecodeCapable/src/vaapimodule.cc
[delete] https://crrev.com/2cfc851f8b6ba6e899418774be8d761b37efe700/client/site_tests/hardware_VideoDecodeCapable/control.vaapi
[delete] https://crrev.com/2cfc851f8b6ba6e899418774be8d761b37efe700/client/site_tests/hardware_VideoDecodeCapable/src/v4l2module.cc
[delete] https://crrev.com/2cfc851f8b6ba6e899418774be8d761b37efe700/client/site_tests/hardware_VideoDecodeCapable/src/Makefile
[delete] https://crrev.com/2cfc851f8b6ba6e899418774be8d761b37efe700/client/site_tests/hardware_VideoDecodeCapable/hardware_VideoDecodeCapable.py
[delete] https://crrev.com/2cfc851f8b6ba6e899418774be8d761b37efe700/client/site_tests/hardware_VideoDecodeCapable/src/setup_vaapi.py
[delete] https://crrev.com/2cfc851f8b6ba6e899418774be8d761b37efe700/client/site_tests/hardware_VideoDecodeCapable/src/setup_vaapi_drm.py
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 25 2016 

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

commit 81f97b2955ce9f0f14cefbffcb2592373f02fb96
Author: Ricky Liang <jcliang@chromium.org>
Date: Mon Nov 21 05:07:41 2016

autotest-tests: Remove hardware_VideoDecodeCapable test

BUG= chromium:665324 
CQ-DEPEND=CL:412306
TEST=None

Change-Id: I83d3e7915acedf4ae2cc77f31ab4e81f7749eeb0
Reviewed-on: https://chromium-review.googlesource.com/413073
Commit-Ready: Ricky Liang <jcliang@chromium.org>
Tested-by: Ricky Liang <jcliang@chromium.org>
Reviewed-by: Owen Lin <owenlin@chromium.org>
Reviewed-by: Wu-cheng Li <wuchengli@chromium.org>

[modify] https://crrev.com/81f97b2955ce9f0f14cefbffcb2592373f02fb96/chromeos-base/autotest-tests/autotest-tests-9999.ebuild

Comment 6 by jcliang@chromium.org, Nov 26 2016

Status: Fixed (was: Started)
►

Sign in to add a comment