Collect information on the dump when GetDC fails.

base/debug/gdi_debug_util_win.h

Index: base/debug/gdi_debug_util_win.h
diff --git a/base/debug/gdi_debug_util_win.h b/base/debug/gdi_debug_util_win.h
index 5887ecb8464184e63013a374fa5d9bb701e207ba..440e54771ac28b7e8520a9e15fcdbacbff0bc020 100644
--- a/base/debug/gdi_debug_util_win.h
+++ b/base/debug/gdi_debug_util_win.h
@@ -13,7 +13,11 @@ namespace base {
 namespace debug {
 
 // Crashes the process leaving valuable information on the dump via
-// debug::alias so we can find what is causing the allocation failures.
+// |base::debug::Alias| so we can find what is causing the GDI failures.

[Peter Kasting, 2016/10/26 17:10:42]

Nit: No || on classnames (just variable names)

+void BASE_EXPORT CollectGDIUsageAndDie();
+
+// Crashes the process leaving valuable information about the bitmap
+// properties so we can find what is causing the bitmap allocation failures.
 void BASE_EXPORT GDIBitmapAllocFailure(BITMAPINFOHEADER* header,
                                        HANDLE shared_section);

[Peter Kasting, 2016/10/26 17:10:42]

Nit: I suggest just one function declaration, to make it clearer these really do
the same thing, and simplify your implementation:

// Crashes the process, using base::debug::Alias to leave valuable debugging
// information in the crash dump.  Pass values for |header| and |shared_section|
// in the event of a bitmap allocation failure, to gather information about
// those as well.
void BASE_EXPORT CollectGDIUsageAndDie(BITMAPINFOHEADER* header = nullptr,
                                       HANDLE shared_section = nullptr);