Add perf tests for starting Chrome with URL

tools/telemetry/telemetry/page/gtest_test_results.py

 # Copyright (c) 2013 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 logging
 import sys
 import time
 import unittest
 
 from telemetry.page import page_test_results
@@ skipping 45 matching lines @@
     super(GTestTestResults, self).addSuccess(test)
     test_name = GTestTestResults._formatTestname(test)
     print >> self._output_stream, '[       OK ]', test_name, (
         '(%0.f ms)' % self._GetMs())
     sys.stdout.flush()
 
   def addSkip(self, test, reason):
     super(GTestTestResults, self).addSkip(test, reason)
     test_name = GTestTestResults._formatTestname(test)
     logging.warning('===== SKIPPING TEST %s: %s =====', test_name, reason)
+    if self._timestamp == None:
+      self._timestamp = time.time()

[jeremy, 2013/12/15 13:03:36]

What is this change needed for?

[aberent, 2013/12/16 17:07:43]

Actually nothing to do with this CL. I hit a crash during testing on the next
line when _GetMs was called, because _GetMs does arithmetic on self.timestamp
(hence crashes if it is None). Not sure why this hasn't been hit before, nor am
I sure if addSkip is ever called after the timestamp is set, but it certainly
can be called before the timestamp is set.

The specific case I had was that I was trying to run record_wpr on a page set
including a local file, and record_wpr crashed when it tried to skip the local
file.

[jeremy, 2013/12/17 11:29:44]

Makes sense, can you ask dtu@ or someone else familiar with this code to review
this part?

     print >> self._output_stream, '[       OK ]', test_name, (
         '(%0.f ms)' % self._GetMs())
     sys.stdout.flush()
 
   def PrintSummary(self):
     unit = 'test' if len(self.successes) == 1 else 'tests'
     print >> self._output_stream, '[  PASSED  ]', (
         '%d %s.' % (len(self.successes), unit))
     if self.errors or self.failures:
       all_errors = self.errors[:]
       all_errors.extend(self.failures)
       unit = 'test' if len(all_errors) == 1 else 'tests'
       print >> self._output_stream, '[  FAILED  ]', (
           '%d %s, listed below:' % (len(all_errors), unit))
       for test, _ in all_errors:
         print >> self._output_stream, '[  FAILED  ] ', (
             GTestTestResults._formatTestname(test))
     if not self.wasSuccessful():
       print >> self._output_stream
       count = len(self.errors) + len(self.failures)
       unit = 'TEST' if count == 1 else 'TESTS'
       print >> self._output_stream, '%d FAILED %s' % (count, unit)
     print >> self._output_stream
     sys.stdout.flush()