[Telemetry] Copy perf_result_data_type.py & perf_tests_results_helper.py to telemetry/util

[Telemetry] Copy perf_result_data_type.py & perf_tests_results_helper.py to telemetry/util

Move perf_tests_helper.py to
telemetry/util/perf_tests_helper.py

BUG=497905
Committed: https://crrev.com/13d4890318c7845e534fd5b786051b50b973c603
Cr-Commit-Position: refs/heads/master@{#333664}

[nednguyen, 2015-06-08 23:35:50]

Patchset #1 (id:1) has been deleted

[nednguyen, 2015-06-08 23:37:15]

nednguyen@google.com changed reviewers:
+ aiolos@chromium.org, dtu@chromium.org

[nednguyen, 2015-06-09 00:26:28]

The CQ bit was checked by nednguyen@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-06-09 00:27:51]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1171803003/20001

[commit-bot: I haz the power, 2015-06-09 02:13:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-09 02:13:45]

Dry run: Try jobs failed on following builders:
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[nednguyen, 2015-06-09 15:12:58]

nednguyen@google.com changed reviewers:
+ sullivan@chromium.org

[nednguyen, 2015-06-10 00:17:28]

On 2015/06/09 15:12:58, nednguyen wrote:

Ping

[aiolos (Not reviewing), 2015-06-10 00:34:40]

nits + lgtm

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/results/buildbot_output_formatter.py (right):

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/results/buildbot_output_formatter.py:5: from telemetry
import value as value_module
This import should be after the perf_tests_helper import.

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/value/histogram.py (right):

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/value/histogram.py:6: from telemetry import value as
value_module
This import should go after perf_tests_helper.

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/web_perf/metrics/smoothness.py (right):

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/web_perf/metrics/smoothness.py:7: from
telemetry.util.perf_tests_helper import FlattenList
Would you mind changing this to a module level import while you're here?

[nednguyen, 2015-06-10 01:18:41]

The CQ bit was checked by nednguyen@google.com

[nednguyen, 2015-06-10 01:18:41]

The patchset sent to the CQ was uploaded after l-g-t-m from aiolos@chromium.org
Link to the patchset: https://codereview.chromium.org/1171803003/#ps40001
(title: "Address Kari's comments")

[nednguyen, 2015-06-10 01:18:47]

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/results/buildbot_output_formatter.py (right):

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/results/buildbot_output_formatter.py:5: from telemetry
import value as value_module
On 2015/06/10 00:34:40, aiolos wrote:
> This import should be after the perf_tests_helper import.

Done.

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/value/histogram.py (right):

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/value/histogram.py:6: from telemetry import value as
value_module
On 2015/06/10 00:34:40, aiolos wrote:
> This import should go after perf_tests_helper.

Done.

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/web_perf/metrics/smoothness.py (right):

https://codereview.chromium.org/1171803003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/web_perf/metrics/smoothness.py:7: from
telemetry.util.perf_tests_helper import FlattenList
On 2015/06/10 00:34:40, aiolos wrote:
> Would you mind changing this to a module level import while you're here?

Done.

[commit-bot: I haz the power, 2015-06-10 01:20:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1171803003/40001

[commit-bot: I haz the power, 2015-06-10 02:50:04]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-06-10 02:51:04]

Patchset 2 (id:??) landed as
https://crrev.com/13d4890318c7845e534fd5b786051b50b973c603
Cr-Commit-Position: refs/heads/master@{#333664}