video_PlaybackPerf reports 0 dropped frames if no frame is decoded |
||
Issue descriptionBetween 71.11067.0.0 and 71.11072.0.0, we run into the regression. Playback is not started and thus no frames is decoded. https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgICQ57_P6QsM https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgICQ15SPswoM video_PlaybackPerf even reports performance values; # of dropped frames is 0, dropped frame rate is 100%, and cpu usage is computed correctly but quite low because no playback is being performed. video_PlaybackPerf should not report any values in the case.
,
Oct 9
Issue 892981 has been merged into this issue.
,
Oct 10
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/c41d75034865205295895fd0d50be844fbf0e360 commit c41d75034865205295895fd0d50be844fbf0e360 Author: Hirokazu Honda <hiroh@chromium.org> Date: Wed Oct 10 04:14:10 2018 video_PlaybackPerf: Fail the test if the number of decoded frames is too small If playback is not started and thus no frames is decoded, video_PlaybackPerf reports the peformance values as follows; the number of dropped frames is 0, dropped frame rate is 100%, cpu usage is computed correctly but quite low due to no playback. video_PlaybackPerf should not report any values and fails in the case. BUG= chromium:891972 TEST=video_PlaybackPerf.{h264, vp8, vp9.4k.60} on panther Change-Id: Iea6cfc124fefcf97b8b3d9ff80d3ca27c983b502 Reviewed-on: https://chromium-review.googlesource.com/1260662 Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com> Tested-by: Hirokazu Honda <hiroh@chromium.org> Reviewed-by: Shuo-Peng Liao <deanliao@google.com> [modify] https://crrev.com/c41d75034865205295895fd0d50be844fbf0e360/client/site_tests/video_PlaybackPerf/video_PlaybackPerf.py
,
Oct 12
Since this change was landed, there looks no failures due to it. Close the issue. |
||
►
Sign in to add a comment |
||
Comment 1 by hiroh@chromium.org
, Oct 4