[PCv1] Avoid using dot in metrics name

tools/perf/measurements/page_cycler.py

 # Copyright 2012 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.
 
 """The page cycler measurement.
 
 This measurement registers a window load handler in which is forces a layout and
 then records the value of performance.now(). This call to now() measures the
 time from navigationStart (immediately after the previous page's beforeunload
 event) until after the layout in the page's load event. In addition, two garbage
 collections are performed in between the page loads (in the beforeunload event).
 This extra garbage collection time is not included in the measurement times.
-
+page_

[nednguyen, 2016/06/02 01:56:06]

typo? :P

[kouhei (in TOK), 2016/06/02 01:59:57]

typo. Ack. Will fix and land

 Finally, various memory and IO statistics are gathered at the very end of
 cycling all pages.
 """
 
 import collections
 import os
 
 from telemetry.core import util
 from telemetry.page import legacy_page_test
 from telemetry.value import scalar
@@ skipping 80 matching lines @@
 
     keychain_metric.KeychainMetric.CustomizeBrowserOptions(options)
 
   def ValidateAndMeasurePage(self, page, tab, results):
     tab.WaitForJavaScriptExpression('__pc_load_time', 60)
 
     chart_name_prefix = ('cold_' if self.IsRunCold(page.url) else
                          'warm_')
 
     results.AddValue(scalar.ScalarValue(
+        results.current_page, '%stimes-page_load_time' % chart_name_prefix,
+        'ms', tab.EvaluateJavaScript('__pc_load_time'),
+        description='Average page load time. Measured from '
+                    'performance.timing.navigationStart until the completion '
+                    'time of a layout after the window.load event. Cold times '
+                    'are the times when the page is loaded cold, i.e. without '
+                    'loading it before, and warm times are times when the '
+                    'page is loaded after being loaded previously.'))
+    results.AddValue(scalar.ScalarValue(
+        results.current_page, '%stimes-time_to_onload' % chart_name_prefix,
+        'ms', tab.EvaluateJavaScript('performance.timing.loadEventStart'
+                                     '- performance.timing.navigationStart'),
+        description='Time to onload. This is temporary metric to check that '
+                    'PCv1 and PCv2 emit similar results'))
+
+    # TODO(kouhei): Remove below. crbug.com/616342
+    results.AddValue(scalar.ScalarValue(
         results.current_page, '%stimes.page_load_time' % chart_name_prefix,
         'ms', tab.EvaluateJavaScript('__pc_load_time'),
         description='Average page load time. Measured from '
                     'performance.timing.navigationStart until the completion '
                     'time of a layout after the window.load event. Cold times '
                     'are the times when the page is loaded cold, i.e. without '
                     'loading it before, and warm times are times when the '
                     'page is loaded after being loaded previously.'))
     results.AddValue(scalar.ScalarValue(
         results.current_page, '%stimes.time_to_onload' % chart_name_prefix,
@@ skipping 29 matching lines @@
     # preserve any initial profile cache for as long as possible.
     # The second is that, if we did cold runs first, we'd have a transition
     # page set during which we wanted the run for each URL to both
     # contribute to the cold data and warm the catch for the following
     # warm run, and clearing the cache before the load of the following
     # URL would eliminate the intended warmup for the previous URL.
     return self._has_loaded_page[url] >= self._cold_run_start_index
 
   def DidRunPage(self, platform):
     self._power_metric.Close()