New issue
Advanced search Search tips

Issue 620564 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Add timestamp testcase in VideoEncodeAccelerator unittest

Project Member Reported by wuchengli@chromium.org, Jun 16 2016

Issue description

The timetsamp testcase is reverted. We should add it back. Let's use this issue to track.
 
Labels: VideoShortList
Labels: -M-54 M-53
Sheng-hao. Can you add the test in M53? 620565 will affect lots of platforms and we need a test to ensure we don't have regressions.
Cc: posciak@chromium.org
Pawel. I have some questions about the unit test:
1. Wu-Cheng mentioned that 1 input frame can be mapped to multiple output frames. Is it possible that 1 input frame is mapped to 0 output frame?
2. Currently we supply timestamp=0 if the driver does not support it. Do we want to allow the test to pass if all output frame timestamps are 0 (for the driver not supported case)?

Status: Started (was: Assigned)
Project Member

Comment 6 by sheriffbot@chromium.org, Jul 4 2016

Labels: -M-53 M-54 MovedFrom-53
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 7 by bugdroid1@chromium.org, Jul 12 2016

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

commit d701f00d4b691011bb7a019ea3a0e2fbd65c0611
Author: shenghao <shenghao@chromium.org>
Date: Tue Jul 12 13:25:16 2016

VEA unit test: Add timestamp test

Verify that one input timestamps match at least one output timestamp.

BUG= 620564 
TEST=Verify that the test pass on minnie

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

[modify] https://crrev.com/d701f00d4b691011bb7a019ea3a0e2fbd65c0611/media/gpu/video_encode_accelerator_unittest.cc

Status: Fixed (was: Started)

Comment 9 by son...@google.com, Nov 16 2016

Labels: -VideoShortList videoshortlist
Status: Verified (was: Fixed)
Verified on build 8743.85.0
All unit tests are passing on minnie.
https://uberchromegw.corp.google.com/i/chromeos/builders/veyron_minnie-release/builds/566

Sign in to add a comment