Telemetry: Check in PrintPerfResult that the flattened list is also non-zero.

Telemetry: Check in PrintPerfResult that the flattened list is also non-zero.

BUG=304335
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=227271

[rmcilroy, 2013-10-04 16:23:10]

I've been seeing errors like the following in the smoothness_measure benchmark:

buildbot_page_measurement_results.py, line 95, in PrintSummary()
html_page_measurement_results.py, line 82, in _PrintPerfResult()
buildbot_page_measurement_results.py, line 20, in _PrintPerfResult()
perf_tests_results_helper.py, line 120, in PrintPerfResult()
perf_tests_results_helper.py, line 79, in _MeanAndStdDevFromList
    value = values[0]
IndexError: list index out of range

Looking through the code, the only way I can see which would cause an index out
of bounds error here would be if the original list only contained other empty
lists (in which case, the original list will have size > 0, while the flattened
list will be empty). Hopefully this would fix such a situation. PTAL.

[tonyg, 2013-10-05 00:34:51]

lgtm

Could you also please mention the newly filed http://crbug.com/304335 in the
BUG= field of the description?

[rmcilroy, 2013-10-07 07:27:42]

> Could you also please mention the newly filed http://crbug.com/304335 in the
> BUG= field of the description?

Done.

Thanks.

[commit-bot: I haz the power, 2013-10-07 07:28:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rmcilroy@chromium.org/25999004/14001

[bulach, 2013-10-07 07:39:04]

late lgtm, sorry for the delay..

[commit-bot: I haz the power, 2013-10-07 08:19:08]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win_x64_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_x64_re...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-10-07 08:33:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rmcilroy@chromium.org/25999004/14001

[commit-bot: I haz the power, 2013-10-07 08:44:28]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-10-07 11:11:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rmcilroy@chromium.org/25999004/14001

[commit-bot: I haz the power, 2013-10-07 11:47:02]

Change committed as 227271