New issue
Advanced search Search tips

Issue 899604 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Dec 6
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 899586



Sign in to add a comment

Migrate video_JpegDecodeAccelerator to Tast

Project Member Reported by dstaessens@google.com, Oct 29

Issue description

The video_JpegDecodeAccelerator test is currently running on the autotest framework. This bug tracks migrating the test to the new Tast framework: go/tast.
 
Blocking: 899586
Cc: henryhsu@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 12

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

commit 55e0048aceedf1b52ffdc3d06d3f8a52383abb39
Author: David Staessens <dstaessens@chromium.org>
Date: Mon Nov 12 04:30:36 2018

media/gpu: Add parameter to JPEG decode test to overwrite default test data dir.

When the HW JPEG decode tests are run by the Tast test framework, the location
of the test file directory will vary. This CL adds the '--test_data_path'
parameter to the test binary, so Tast can overwrite the default test file path.

TEST=ran JPEG decode tests on eve

BUG= 899604 

Change-Id: I7f242eb4356445a3bc3bb44a9d40221009343c7e
Reviewed-on: https://chromium-review.googlesource.com/c/1328063
Commit-Queue: David Staessens <dstaessens@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#607142}
[modify] https://crrev.com/55e0048aceedf1b52ffdc3d06d3f8a52383abb39/media/gpu/jpeg_decode_accelerator_unittest.cc
[modify] https://crrev.com/55e0048aceedf1b52ffdc3d06d3f8a52383abb39/media/gpu/vaapi/vaapi_jpeg_decode_accelerator_unittest.cc

Project Member

Comment 4 by bugdroid1@chromium.org, Nov 16

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/tast-tests/+/30d88aff8d23695ad6a8477b8681bc32195973ee

commit 30d88aff8d23695ad6a8477b8681bc32195973ee
Author: David Staessens <dstaessens@chromium.org>
Date: Fri Nov 16 02:49:23 2018

video.DecodeAccelJpeg: Add JPEG HW decode accelerator tests to Tast.

TEST=DecodeAccelJpeg

CQ-DEPEND=CL:1328403

BUG= chromium:899604 

Change-Id: I37f12c0437818bb5a16e106446af6d854e3e4347
Reviewed-on: https://chromium-review.googlesource.com/1328384
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: David Staessens <dstaessens@chromium.org>
Reviewed-by: Shuhei Takahashi <nya@chromium.org>

[add] https://crrev.com/30d88aff8d23695ad6a8477b8681bc32195973ee/src/chromiumos/tast/local/bundles/cros/video/decode_accel_jpeg.go
[modify] https://crrev.com/30d88aff8d23695ad6a8477b8681bc32195973ee/src/chromiumos/tast/local/bundles/cros/video/lib/chrometest/chrometest.go

Project Member

Comment 5 by bugdroid1@chromium.org, Nov 16

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

commit ea057bbcad56c48b4604ca1bab292b7a1ec7fb55
Author: David Staessens <dstaessens@chromium.org>
Date: Fri Nov 16 02:49:23 2018

tast-local-tests-cros: Add files used by the jpeg decode tests.

> File sizes:
peach_pi-1280x720.jpg	136K
peach_pi-40x23.jpg	1K
peach_pi-41x22.jpg	1K
peach_pi-41x23.jpg	1K
pixel-1280x720.jpg	119K

TEST=DecodeAccelJpeg

BUG= chromium:899604 

Change-Id: Iaa3cc567a01fe5608ea135eefbfd859219e03631
Reviewed-on: https://chromium-review.googlesource.com/1328403
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: David Staessens <dstaessens@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Shuhei Takahashi <nya@chromium.org>

[modify] https://crrev.com/ea057bbcad56c48b4604ca1bab292b7a1ec7fb55/chromeos-base/tast-local-tests-cros/tast-local-tests-cros-9999.ebuild
[modify] https://crrev.com/ea057bbcad56c48b4604ca1bab292b7a1ec7fb55/chromeos-base/tast-local-tests-cros/Manifest
[modify] https://crrev.com/ea057bbcad56c48b4604ca1bab292b7a1ec7fb55/chromeos-base/tast-local-tests-cros/files/external_data.conf

Components: OS>Kernel>Video
Status: Verified (was: Assigned)

Sign in to add a comment