telemetry: Report error correctly on connection timeout

tools/telemetry/telemetry/page/page_runner.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 logging
 import optparse
 import os
 import random
 import sys
 import time
@@ skipping 272 matching lines @@
       if finder_options.profiler:
         state.StopProfiling()
 
       if (test.StopBrowserAfterPage(state.browser, page)):
         state.StopBrowser()
 
       return
     except exceptions.BrowserGoneException as e:
       state.StopBrowser()
       if attempt_num == max_attempts:
-        logging.error('Aborting after too many retries')
-        raise
-      if test.is_multi_tab_test:
+        results.AddValue(failure.FailureValue.FromMessage(
+            page, 'Failed to connect to browser after too many retries.'))
+      elif test.is_multi_tab_test:
         logging.error('Aborting multi-tab test after browser crashed')
         raise
-      logging.warning(str(e))
+      else:
+        logging.warning(str(e))
 
 
 @decorators.Cache
 def _UpdatePageSetArchivesIfChanged(page_set):
   # Scan every serving directory for .sha1 files
   # and download them from Cloud Storage. Assume all data is public.
   all_serving_dirs = page_set.serving_dirs.copy()
   # Add individual page dirs to all serving dirs.
   for page in page_set:
     if page.is_file:
@@ skipping 220 matching lines @@
     logging.warning('Device is thermally throttled before running '
                     'performance tests, results will vary.')
 
 
 def _CheckThermalThrottling(platform):
   if not platform.CanMonitorThermalThrottling():
     return
   if platform.HasBeenThermallyThrottled():
     logging.warning('Device has been thermally throttled during '
                     'performance tests, results will vary.')