Retry tests more agressively in nacl_integration.

ppapi/native_client/tools/browser_tester/browser_tester.py

Index: ppapi/native_client/tools/browser_tester/browser_tester.py
diff --git a/ppapi/native_client/tools/browser_tester/browser_tester.py b/ppapi/native_client/tools/browser_tester/browser_tester.py
index e22bfde2994cd2d0d9ab5044ea176905257e3a60..777fd8e985d10505ebd4485221a33012f016c291 100755
--- a/ppapi/native_client/tools/browser_tester/browser_tester.py
+++ b/ppapi/native_client/tools/browser_tester/browser_tester.py
@@ -298,6 +298,8 @@ def RunTestsOnce(url, options):
         else:
           err += '\nThe test probably did not get a callback that it expected.'
         listener.ServerError(err)
+        if not server.received_request:
+          raise RetryTest('Chrome hung before running the test.')
         break
       elif not options.interactive and HardTimeout(options.hard_timeout):
         listener.ServerError('The test took over %.1f seconds.  This is '
@@ -356,6 +358,8 @@ def Run(url, options):
   while True:
     try:
       result = RunTestsOnce(url, options)
+      if result and options.tool is None:

[Mark Seaborn, 2013/11/14 01:27:58]

To be honest, I don't understand why you're adding two new 'raise RetryTest'
cases.  Doesn't this one retry on all failures, so that the first wouldn't be
necessary?

Also, I don't understand why you check options.tool here but not above.

+        raise RetryTest('HACK retrying failed test.')
       break
     except RetryTest:
       # Only retry once.