[Android] Extend setup timeout for content_browsertests.

build/android/pylib/browsertests/dispatch.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 os
 
 from pylib import android_commands
 from pylib import cmd_helper
 from pylib import constants
@@ skipping 44 matching lines @@
 
     # Get tests and split them up based on the number of devices.
   if options.gtest_filter:
     all_tests = [t for t in options.gtest_filter.split(':') if t]
   else:
     all_enabled = gtest_dispatch.GetAllEnabledTests(RunnerFactory,
                                                     attached_devices)
     all_tests = _FilterTests(all_enabled)
 
   # Run tests.
+  # TODO(nileshagrawal): remove this abnormally long setup timeout once fewer
+  # files are pushed to the devices for content_browsertests: crbug.com/138275
+  setup_timeout = 20 * 60  # 20 minutes
   test_results = shard.ShardAndRunTests(RunnerFactory, attached_devices,
                                         all_tests, options.build_type,
+                                        setup_timeout=setup_timeout,
                                         test_timeout=None)

[nilesh, 2013/04/05 21:23:54]

we also need more than 7 mins to run these, specially for bots with only 1
device. 

[newt (away), 2013/04/05 21:41:25]

test_timeout=None means they can't time out. see:
build/android/pylib/utils/watchdog_timer.py

[craigdh, 2013/04/05 22:00:03]

Both content_browsertests and gtests have timeouts disabled during the tests
because of how they use a gtest filter to run multiple tests together.
Instrumentation tests reset the timeout after each test.

[craigdh, 2013/04/05 22:18:37]

In that case please take out test_timeout=None, I assume none of the individual
tests take more than seven minutes?

   report_results.LogFull(
       results=test_results,
       test_type='Unit test',
       test_package=constants.BROWSERTEST_SUITE_NAME,
       build_type=options.build_type,
       flakiness_server=options.flakiness_dashboard_server)
   report_results.PrintAnnotation(test_results)
 
 def _FilterTests(all_enabled_tests):
   """Filters out tests and fixtures starting with PRE_ and MANUAL_."""
   return [t for t in all_enabled_tests if _ShouldRunOnBot(t)]
 
 def _ShouldRunOnBot(test):
   fixture, case = test.split('.', 1)
   if _StartsWith(fixture, case, "PRE_"):
     return False
   if _StartsWith(fixture, case, "MANUAL_"):
     return False
   return True
 
 def _StartsWith(a, b, prefix):
   return a.startswith(prefix) or b.startswith(prefix)