Add perf tests for starting Chrome with URL

tools/telemetry/telemetry/page/gtest_test_results.py

Index: tools/telemetry/telemetry/page/gtest_test_results.py
diff --git a/tools/telemetry/telemetry/page/gtest_test_results.py b/tools/telemetry/telemetry/page/gtest_test_results.py
index ff1b115968f97c9cc32b9c87bfc3bc7035e91ebc..69b0002081e75d8cd56265f61f80650b8b94c7a2 100644
--- a/tools/telemetry/telemetry/page/gtest_test_results.py
+++ b/tools/telemetry/telemetry/page/gtest_test_results.py
@@ -63,6 +63,8 @@ class GTestTestResults(page_test_results.PageTestResults):
     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()