Sanitize event names before adding to results object

tools/perf/metrics/timeline_unittest.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import unittest
 
 from metrics import timeline
 from telemetry.core.timeline import model as model_module
 from telemetry.page import page as page_module
 from telemetry.page import page_measurement_results
@@ skipping 15 matching lines @@
 
   def AssertHasPageSpecificScalarValue(self, name, units, expected_value):
     value = self.GetPageSpecificValueNamed(name)
     self.test.assertEquals(units, value.units)
     self.test.assertTrue(isinstance(value, scalar.ScalarValue))
     self.test.assertEquals(expected_value, value.value)
 
   def __str__(self):
     return '\n'.join([repr(x) for x in self.all_page_specific_values])
 
+class LoadTimesTimelineMetric(unittest.TestCase):
+  def GetResultsForModel(self, metric, model):
+    metric.model = model
+    results = TestPageMeasurementResults(self)
+    tab = None
+    metric.AddResults(tab, results)
+    return results
+
+  def testSanitizing(self):
+    model = model_module.TimelineModel()
+    renderer_main = model.GetOrCreateProcess(1).GetOrCreateThread(2)
+    renderer_main.name = 'CrRendererMain'
+
+    # [      X       ]
+    #      [  Y  ]
+    renderer_main.BeginSlice('cat1', 'x.y', 10, 0)
+    renderer_main.EndSlice(20, 20)
+    model.FinalizeImport()
+
+    metric = timeline.LoadTimesTimelineMetric(timeline.TRACING_MODE)
+    metric.renderer_process = renderer_main.parent
+    results = self.GetResultsForModel(metric, model)
+    results.AssertHasPageSpecificScalarValue(
+      'CrRendererMain|x_y', 'ms', 10)
+    results.AssertHasPageSpecificScalarValue(
+      'CrRendererMain|x_y_max', 'ms', 10)
+    results.AssertHasPageSpecificScalarValue(
+      'CrRendererMain|x_y_avg', 'ms', 10)
+
+
 class ThreadTimesTimelineMetricUnittest(unittest.TestCase):
   def GetResultsForModel(self, metric, model):
     metric.model = model
     results = TestPageMeasurementResults(self)
     tab = None
     metric.AddResults(tab, results)
     return results
 
   def testBasic(self):
     model = model_module.TimelineModel()
@@ skipping 36 matching lines @@
     metric = timeline.ThreadTimesTimelineMetric()
     metric.report_renderer_main_details = True
     results = self.GetResultsForModel(metric, model)
 
     results.AssertHasPageSpecificScalarValue(
       'thread_renderer_main_clock_time_percentage', '%', 75)
     results.AssertHasPageSpecificScalarValue(
       'renderer_main|cat', 'ms', 30)
     results.AssertHasPageSpecificScalarValue(
       'renderer_main|idle', 'ms', 10)