diff options
| author | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-11 00:26:04 +0000 |
|---|---|---|
| committer | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-11 00:26:04 +0000 |
| commit | ca7bf5806cf02793cbd756b7138cd766cc2caf28 (patch) | |
| tree | a94a2500f383250f0fdb1d0470949d44c295e4fd /chrome/browser | |
| parent | 93b552eb1179e797613e64e4dd7eac4ac8fbb8bd (diff) | |
| download | chromium_src-ca7bf5806cf02793cbd756b7138cd766cc2caf28.zip chromium_src-ca7bf5806cf02793cbd756b7138cd766cc2caf28.tar.gz chromium_src-ca7bf5806cf02793cbd756b7138cd766cc2caf28.tar.bz2 | |
gtk: Post the SendNotification in a task on the UI thread. In rare instances, SendNotifications was being run before CancelAuthDeferred, so the proper cleanup was not happening before this object was deleted which leads to a crash.
BUG=18973
TEST=Instructions in the bug report.
Review URL: http://codereview.chromium.org/165268
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22984 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
| -rw-r--r-- | chrome/browser/login_prompt_gtk.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/chrome/browser/login_prompt_gtk.cc b/chrome/browser/login_prompt_gtk.cc index ef98d26..f38be97 100644 --- a/chrome/browser/login_prompt_gtk.cc +++ b/chrome/browser/login_prompt_gtk.cc @@ -175,7 +175,8 @@ class LoginHandlerGtk : public LoginHandler, if (!WasAuthHandled(true)) { request_loop_->PostTask(FROM_HERE, NewRunnableMethod( this, &LoginHandlerGtk::CancelAuthDeferred)); - SendNotifications(); + ui_loop_->PostTask(FROM_HERE, NewRunnableMethod( + this, &LoginHandlerGtk::SendNotifications)); } // Delete this object once all InvokeLaters have been called. |