x11: Unset the X11 error handlers when the message-loop starts getting destroyed.

x11: Unset the X11 error handlers when the message-loop starts getting destroyed.

The X11 error handlers report the errors from a PostTask()ed function,
but since the X11 error may have happened when the message-loop was
already getting destroyed (e.g. the message-pump destructor), this
can cause errors.

BUG=146928
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=157056

[sadrul, 2012-09-16 16:32:33]

http://codereview.chromium.org/10933113/diff/1/chrome/browser/chrome_browser_...
File chrome/browser/chrome_browser_main.cc (right):

http://codereview.chromium.org/10933113/diff/1/chrome/browser/chrome_browser_...
chrome/browser/chrome_browser_main.cc:1478: UnsetBrowserX11ErrorHandlers();
I considered doing this from
ChromeBrowserMainPartsLinux::PostMainMessageLoopRun. However, since the
X11-error handlers are set in ChromeBrowserMainParts (line 1064 above), I added
this here instead.

[sky, 2012-09-16 18:48:06]

LGTM