[Telemetry]: Assert that page measurement results don't contain a '.' in their name.

[Telemetry]: Assert that page measurement results don't contain a '.' in their name.

Telemetry uses a '.' to delimit chart_name and trace_names in Value types, so
trace names cannot include a '.' in them.

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

[rmcilroy, 2014-01-06 11:18:40]

Nat, please review this CL.

[tonyg, 2014-01-06 15:41:25]

lgtm

https://codereview.chromium.org/116043006/diff/1/tools/telemetry/telemetry/pa...
File tools/telemetry/telemetry/page/page_measurement_results.py (right):

https://codereview.chromium.org/116043006/diff/1/tools/telemetry/telemetry/pa...
tools/telemetry/telemetry/page/page_measurement_results.py:63: assert('.' not in
trace_name)
Drop the parens and move the comment into an error message. Eg.

assert '.' not in trace_name, 'Trace names cannot contain "." as this...'

[rmcilroy, 2014-01-06 16:30:43]

Thanks Tony!

https://codereview.chromium.org/116043006/diff/1/tools/telemetry/telemetry/pa...
File tools/telemetry/telemetry/page/page_measurement_results.py (right):

https://codereview.chromium.org/116043006/diff/1/tools/telemetry/telemetry/pa...
tools/telemetry/telemetry/page/page_measurement_results.py:63: assert('.' not in
trace_name)
On 2014/01/06 15:41:25, tonyg wrote:
> Drop the parens and move the comment into an error message. Eg.
> 
> assert '.' not in trace_name, 'Trace names cannot contain "." as this...'

Done.

[commit-bot: I haz the power, 2014-01-06 16:31:09]

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

[commit-bot: I haz the power, 2014-01-06 22:24:58]

Change committed as 243186