Change X11 error handler override to allow easy X11 error checking.

ui/base/x/x11_util_internal.h

Index: ui/base/x/x11_util_internal.h
diff --git a/ui/base/x/x11_util_internal.h b/ui/base/x/x11_util_internal.h
index 70eb088c786e8668e48eb2f65f14f0d0a425a423..7415a2ef6be6c8b61a71ba3f7f86a55f8f0ed1c4 100644
--- a/ui/base/x/x11_util_internal.h
+++ b/ui/base/x/x11_util_internal.h
@@ -50,6 +50,12 @@ UI_EXPORT void SetX11ErrorHandlers(XErrorHandler error_handler,
 UI_EXPORT void LogErrorEventDescription(Display* dpy,
                                         const XErrorEvent& error_event);
 
+// CHECK that no X11 errors have been reported. Uses XSync to ensure that all
+// requests have been received and processed by the X server. Uses the X
+// display contained in the reported XErrorEvent. Returns true if an X11 error
+// has been reported.

[Ami GONE FROM CHROMIUM, 2011/09/22 17:17:50]

Wait, what?
The comment & function name still say check.

[dominich, 2011/09/22 18:24:12]

Changed the function name and updated the comment. I removed the return value
too as I think it confuses things rather than adds flexibility.

+UI_EXPORT bool CheckFailOnReportedX11Error();
+
 }  // namespace ui
 
 #endif  // UI_BASE_X_X11_UTIL_INTERNAL_H_