Collect information on the dump when GetDC fails.

base/debug/gdi_debug_util_win.h

 // Copyright 2014 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.
 
 #ifndef BASE_DEBUG_GDI_DEBUG_UTIL_WIN_H_
 #define BASE_DEBUG_GDI_DEBUG_UTIL_WIN_H_
 
 #include <windows.h>
 
 #include "base/base_export.h"
 
 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);

 
 }  // namespace debug
 }  // namespace base
 
 #endif  // BASE_DEBUG_GDI_DEBUG_UTIL_WIN_H_