TestShell windows: add --crash-dumps-fulldump option

TestShell windows: add --crash-dumps-fulldump option
to allow the dump including the heap memory.

TEST=none
BUG=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=67925

[gmorrita, 2010-11-30 05:27:57]

Hi tony, could you take a look?
This flag is added to investigate the test shell crash.

[tony, 2010-11-30 17:58:09]

Do you plan on running this on the bots?  It seems like even for one run, this
will generate a lot of data.

Also, I plan on trying to switch from test_shell to DRT sometime this week.

http://codereview.chromium.org/5262011/diff/1/webkit/tools/test_shell/test_sh...
File webkit/tools/test_shell/test_shell_win.cc (right):

http://codereview.chromium.org/5262011/diff/1/webkit/tools/test_shell/test_sh...
webkit/tools/test_shell/test_shell_win.cc:189: new
google_breakpad::ExceptionHandler(dir, 0, &MinidumpCallback, 0, true,
Nit: 80 cols

http://codereview.chromium.org/5262011/diff/1/webkit/tools/test_shell/test_sh...
webkit/tools/test_shell/test_shell_win.cc:192: new
google_breakpad::ExceptionHandler(dir, 0, &MinidumpCallback, 0, true);
Nit: 80 cols

[gmorrita, 2010-12-01 03:36:18]

Hi tony, thank you for reviwiewing!

This patch is not for a bot. 
I made this change when I was investigating a crash bug. 
This was useful for handling crashes on non-dev machine. 
In my case, it's an instance of XP-mode VM.

DRT switch is great! I'm tired to duplicate DRT changes to test shell :D
Breakpad integration also might be helpful for DRT.

http://codereview.chromium.org/5262011/diff/1/webkit/tools/test_shell/test_sh...
File webkit/tools/test_shell/test_shell_win.cc (right):

http://codereview.chromium.org/5262011/diff/1/webkit/tools/test_shell/test_sh...
webkit/tools/test_shell/test_shell_win.cc:189: new
google_breakpad::ExceptionHandler(dir, 0, &MinidumpCallback, 0, true,
On 2010/11/30 17:58:09, tony wrote:
> Nit: 80 cols

Done.

http://codereview.chromium.org/5262011/diff/1/webkit/tools/test_shell/test_sh...
webkit/tools/test_shell/test_shell_win.cc:192: new
google_breakpad::ExceptionHandler(dir, 0, &MinidumpCallback, 0, true);
On 2010/11/30 17:58:09, tony wrote:
> Nit: 80 cols

Done.

[tony, 2010-12-01 18:09:54]

LGTM.  Do you need someone to commit this for you?

[gmorrita, 2010-12-02 00:40:13]

On 2010/12/01 18:09:54, tony wrote:
> LGTM.  Do you need someone to commit this for you?
Thanks! I got a commit right for the gardening so I can commit this by myself.