Breakpad: Disable browser process crash test for one case on 32-bit Windows

ppapi/native_client/tests/breakpad_crash_test/nacl.scons

Index: ppapi/native_client/tests/breakpad_crash_test/nacl.scons
diff --git a/ppapi/native_client/tests/breakpad_crash_test/nacl.scons b/ppapi/native_client/tests/breakpad_crash_test/nacl.scons
index 42f6e4a49f8a26d54821a8c86a534948626ec3a5..06c463205bcce454be1dfccb52cc497cbf9cc224 100644
--- a/ppapi/native_client/tests/breakpad_crash_test/nacl.scons
+++ b/ppapi/native_client/tests/breakpad_crash_test/nacl.scons
@@ -3,6 +3,8 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+import os
+
 Import('env')
 
 if env.Bit('host_windows') or env.Bit('host_mac'):
@@ -40,8 +42,20 @@ if env.Bit('host_windows') or env.Bit('host_mac'):
                             '--expected_process_type=browser'])
   env.AddNodeToTestSuite(
       node, ['chrome_browser_tests'], 'run_breakpad_browser_process_crash_test',
-      is_broken=env.PPAPIBrowserTesterIsBroken() or
-                 env.Bit('running_on_valgrind'))
+      # For some strange reason, this test fails on the second run
+      # (for the SRPC PPAPI proxy) on the Windows XP (32-bit)
+      # buildbots.  The failures modes are that the browser process
+      # hangs, or that a .dmp file is produced with no .txt file.
+      # Maybe there is a problem with the command line getting too
+      # long?
+      # TODO(mseaborn): Remove this when the "--enable-srpc-proxy"
+      # option is removed, or investigate and fix.
+      is_broken=(env.PPAPIBrowserTesterIsBroken() or
+                 env.Bit('running_on_valgrind') or
+                 (env.Bit('host_windows') and
+                  env.Bit('build_x86_32') and
+                  '--enable-srpc-proxy' in
+                      os.environ.get('NACL_BROWSER_FLAGS', ''))))
 
 # This crash in trusted code should produce a crash dump.
 crash_test_url = 'trusted_crash_in_startup.html'