Add timestamp testcase in VideoEncodeAccelerator unittest |
|||||||
Issue descriptionThe timetsamp testcase is reverted. We should add it back. Let's use this issue to track.
,
Jun 28 2016
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.
,
Jun 28 2016
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)?
,
Jun 29 2016
CL uploaded: https://codereview.chromium.org/2104883005/
,
Jun 29 2016
,
Jul 4 2016
Moving this nonessential bug to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
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
,
Jul 12 2016
,
Nov 16 2016
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 |
|||||||
Comment 1 by posciak@chromium.org
, Jun 16 2016