Fix ChromeFrame net tests hang on the windows try bots

chrome_frame/test/net/fake_external_tab.cc

 // Copyright (c) 2011 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.
 
 #include "chrome_frame/test/net/fake_external_tab.h"
 
 #include <atlbase.h>
 #include <atlcom.h>
 #include <exdisp.h>
 
@@ skipping 560 matching lines @@
   watchdog.AddObserver(&credentials, "Windows Security", "");
   testing::InitGoogleTest(&argc, argv);
   FilterDisabledTests();
   test_suite.RunMainUIThread();
 
   if (crash_service)
     base::KillProcess(crash_service, 0, false);
 
   base::KillProcesses(chrome_frame_test::kIEImageName, 0, NULL);
   base::KillProcesses(chrome_frame_test::kIEBrokerImageName, 0, NULL);
+  // Avoid CRT cleanup in debug test runs to ensure that webkit ASSERTs which
+  // check if globals are created and destroyed on the same thread don't fire.
+  // Webkit global objects are created on the inproc renderer thread.
+#ifdef _DEBUG

[M-A Ruel, 2011/08/16 22:03:47]

In general we use #if !defined(NDEBUG) since _DEBUG is a MS-extension. Even if
this is a windows-only test, I'd rather not encourage that.

[ananta, 2011/08/16 22:10:59]

Done.

+  ExitProcess(test_suite.test_result());
+#endif  // _DEBUG
   return test_suite.test_result();
 }