Sanitize event names before adding to results object

tools/perf/metrics/timeline.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 collections
 
 from metrics import Metric
 
 TRACING_MODE = 'tracing-mode'
 TIMELINE_MODE = 'timeline-mode'
 
@@ skipping 36 matching lines @@
     return self._model
 
   @model.setter
   def model(self, model):
     self._model = model
 
   @property
   def renderer_process(self):
     return self._renderer_process
 
+  @renderer_process.setter
+  def renderer_process(self, p):
+    self._renderer_process = p
+
   def AddResults(self, tab, results):
     return
 
 class LoadTimesTimelineMetric(TimelineMetric):
   def __init__(self, mode):
     super(LoadTimesTimelineMetric, self).__init__(mode)
     self.report_main_thread_only = True
 
   def AddResults(self, _, results):
     assert self._model
@@ skipping 15 matching lines @@
       thread_name = thread.name.replace('/','_')
       events = thread.all_slices
 
       for e in events:
         events_by_name[e.name].append(e)
 
       for event_name, event_group in events_by_name.iteritems():
         times = [event.self_time for event in event_group]
         total = sum(times)
         biggest_jank = max(times)
-        full_name = thread_name + '|' + event_name
+
+        # Results objects cannot contain the '.' character, so remove that here.
+        sanitized_event_name = event_name.replace('.', '_')
+
+        full_name = thread_name + '|' + sanitized_event_name
         results.Add(full_name, 'ms', total)
         results.Add(full_name + '_max', 'ms', biggest_jank)
         results.Add(full_name + '_avg', 'ms', total / len(times))
 
     for counter_name, counter in self.renderer_process.counters.iteritems():
       total = sum(counter.totals)
       results.Add(counter_name, 'count', total)
       results.Add(counter_name + '_avg', 'count', total / len(counter.totals))
 
 
@@ skipping 120 matching lines @@
                             self._model.bounds.bounds) * 100
       results.Add(cpu_time_report_name, '%', time_as_percentage)
 
     # TOOD(nduca): When generic results objects are done, this special case
     # can be replaced with a generic UI feature.
     for thread_category, results_for_thread_category in (
         results_per_thread_category.iteritems()):
       if (thread_category == 'renderer_main' and
           self.report_renderer_main_details):
         results_for_thread_category.AddDetailedResults(thread_category, results)