Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(467)

Unified Diff: ui/base/x/x11_util_internal.h

Issue 7889040: Change X11 error handler override to allow easy X11 error checking. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Clean up and remove spurious undefs Created 9 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..b7df4bdbf9666644914ff317ae4fcfa814c1b20a 100644
--- a/ui/base/x/x11_util_internal.h
+++ b/ui/base/x/x11_util_internal.h
@@ -50,6 +50,11 @@ UI_EXPORT void SetX11ErrorHandlers(XErrorHandler error_handler,
UI_EXPORT void LogErrorEventDescription(Display* dpy,
const XErrorEvent& error_event);
+// CHECK thas no X11 errors have been reported. Uses XSync to ensure that all
Ami GONE FROM CHROMIUM 2011/09/20 23:45:51 s/thas/that/
dominich 2011/09/21 19:47:30 Done.
+// requests have been received and processed by the X server. Uses the default X
+// display.
+void CheckFailOnX11Error();
Ami GONE FROM CHROMIUM 2011/09/20 23:45:51 s/On/OnReported/? (the current name makes me think
dominich 2011/09/21 19:47:30 Done.
+
} // namespace ui
#endif // UI_BASE_X_X11_UTIL_INTERNAL_H_

Powered by Google App Engine
This is Rietveld 408576698