Merge 14891 - A better fix for http://www.crbug.com/2044: crash...

Merge 14891 - A better fix for http://www.crbug.com/2044:  crash
on large <canvas> elements.  We disable the __debugbreak
only when skia tells us it is prepared to correctly 
handle a failed (NULL) malloc().  It does this
by calling sk_malloc_flags() without SK_MALLOC_THROW.

Note that, since the switch to tcmalloc, the new_handler
was not getting called at all (since tcmalloc doesn't 
support it yet), so this crash is currently unreproducible
in trunk.  In order to test this change, I reverted the 
tcmalloc change in my client.  This is not the case in the 
stable branch, since it doesn't use tcmalloc, so this change 
is still needed there.  (It will also be needed in trunk 
again once mbelshe's reimplementation of the new_handler
is in).

BUG=http://www.crbug.com/2044
Review URL: http://codereview.chromium.org/100163

TBR=senorblanco@chromium.org

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

[Stephen White, 2009-04-29 22:24:53]

In the Stable branch (which I think is where we are?), in chrome_dll_main.cc,
the "return" should be "return 0".