Fix ChromeFrame net tests hang on the windows try bots

Ensure that webkit ASSERTION's don't fire during CRT cleanup in debug runs of the chrome frame net tests suite by
terminating the exe using ExitProcess in debug builds.

These cause the exe to crash during exit, thus turning the windows try bot runs red.

BUG=none
TEST=chrome frame net tests should run to completion without crashing in debug runs.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=97041

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

lgtm with nit. It would have been nice to not have the assert fire in the first
place. :/

http://codereview.chromium.org/7663015/diff/5001/chrome_frame/test/net/fake_e...
File chrome_frame/test/net/fake_external_tab.cc (right):

http://codereview.chromium.org/7663015/diff/5001/chrome_frame/test/net/fake_e...
chrome_frame/test/net/fake_external_tab.cc:584: #ifdef _DEBUG
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:52]

On 2011/08/16 22:03:47, Marc-Antoine Ruel wrote:
> lgtm with nit. It would have been nice to not have the assert fire in the
first
> place. :/
Agreed. The chrome frame net tests suite basically runs as chrome
--single-process which causes these objects to get created and destroyed on
different threads. 
> 
>
http://codereview.chromium.org/7663015/diff/5001/chrome_frame/test/net/fake_e...
> File chrome_frame/test/net/fake_external_tab.cc (right):
> 
>
http://codereview.chromium.org/7663015/diff/5001/chrome_frame/test/net/fake_e...
> chrome_frame/test/net/fake_external_tab.cc:584: #ifdef _DEBUG
> 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]

http://codereview.chromium.org/7663015/diff/5001/chrome_frame/test/net/fake_e...
File chrome_frame/test/net/fake_external_tab.cc (right):

http://codereview.chromium.org/7663015/diff/5001/chrome_frame/test/net/fake_e...
chrome_frame/test/net/fake_external_tab.cc:584: #ifdef _DEBUG
On 2011/08/16 22:03:47, Marc-Antoine Ruel wrote:
> 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.

Done.