New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 694372 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocked on:
issue 691464



Sign in to add a comment

Enable Thumbnail test on ELM after the bug in imgddk has been fixed.

Project Member Reported by owenlin@chromium.org, Feb 21 2017

Issue description

We disabled the Thumbnail test on ELM for https://bugs.chromium.org/p/chromium/issues/detail?id=691464#c2

After it getting fixed. We should enable it again.
 
Labels: videoshortlist
Owen. According to b/35648505, the issue on elm is fixed in R58-9310.0.0. Can you try re-enabling the test? Thanks.
We still need to update the md5 check sums first:

https://codereview.chromium.org/2725123002
Owen. Can we enable it now?
Project Member

Comment 7 by bugdroid1@chromium.org, Mar 7 2017

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

commit 7c14a585ef7d98436f3f4eb72145245821cbe2d6
Author: Owen Lin <owenlin@chromium.org>
Date: Tue Mar 07 10:32:40 2017

Revert "video_VideoDecodeAccelerator: Skipped the thumbnail test on ELM."

This reverts commit 3171948f74e9f5ffab0d31e928d14524cb4fcc20.

Reason for revert: The issue has been fixed on ELM.

Original change's description:
> video_VideoDecodeAccelerator: Skipped the thumbnail test on ELM.
>
> Before chromium:691464 getting fixed, skip the thumbnail test on ELM so
> other breakages won't get ignored.
>
> BUG= chromium:691464 
> TEST=Run the test on ELM.
>
> Change-Id: Iecda21ec376162b8daa68951d8df94735174522c
> Reviewed-on: https://chromium-review.googlesource.com/445417
> Commit-Ready: Owen Lin <owenlin@chromium.org>
> Tested-by: Owen Lin <owenlin@chromium.org>
> Reviewed-by: Pawel Osciak <posciak@chromium.org>
> Reviewed-by: Wu-cheng Li <wuchengli@chromium.org>
>

TBR=posciak@chromium.org,wuchengli@chromium.org,owenlin@google.com,owenlin@chromium.org
BUG= chromium:694372 

Change-Id: I1f2286824c4b94424a0747f8805de75437f10460
Reviewed-on: https://chromium-review.googlesource.com/450200
Commit-Ready: Owen Lin <owenlin@chromium.org>
Tested-by: Owen Lin <owenlin@chromium.org>
Reviewed-by: Pawel Osciak <posciak@chromium.org>

[modify] https://crrev.com/7c14a585ef7d98436f3f4eb72145245821cbe2d6/client/site_tests/video_VideoDecodeAccelerator/video_VideoDecodeAccelerator.py

Status: Fixed (was: Assigned)

Comment 9 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Yes, it's a different issue. Please file another bug. Thanks.
Status: Verified (was: Fixed)
Closing this issue, new issue is  crbug.com/734707  

Sign in to add a comment