tree d62db17971b6abd2a58b5da5b909d6f1922a0e3a
parent baf2d668da321bb4623df391e37d93ca6bc019bd
author Denis Tosic <dtosic@chromium.org> 1542721167 +0100
committer chrome-bot <chrome-bot@chromium.org> 1542942471 -0800

[autotest][cfm] Fixes and extends metrics gathering for the perf test

The test did not collect all metrics and some metrics failed to upload
due to internally raised exceptions (so the resulted in [0]).

Supporting changes:
- removed _get_value() and instead return `None` if an elem is not found
  - this resolved the bug that caused some metrics to upload [0]
although they had data
- removed tracking of max() values - a single max value was not really
useful
  - a single spike could introduce flakiness and since we anyways upload
the full data
    set the range is visible in the perf dashboards
- removed data sets that were specific to the jmi data format
  -  since they were not available anymore we always uploaded 0
- renamed video_out_res and video_in_res to
vid_out_frame_height/vid_in_frame_height
- system stats as well as encode time is now properly tracked

The data gathering and upload logic will be moved to a new class in
a follow up cl/ to allow for reuse in the actions module.

BUG=b:119806655
TEST=local device

Change-Id: Ia5575a49e6325aedc2258ed9c290c73360adc5c9
Reviewed-on: https://chromium-review.googlesource.com/1343969
Commit-Ready: Denis Tosic <dtosic@google.com>
Tested-by: Denis Tosic <dtosic@google.com>
Reviewed-by: Kristoffer Erlandsson <kerl@google.com>
Reviewed-by: Denis Tosic <dtosic@google.com>