Updates:
        Summary: Linux crash on wrench -> exit

Comment #8 on issue 8712 by [email protected]: Linux crash on wrench  
-> exit
http://code.google.com/p/chromium/issues/detail?id=8712

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=11628

------------------------------------------------------------------------
r11628 | [email protected] | 2009-03-13 06:34:11 -0700 (Fri, 13 Mar 2009)  
| 10 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_window_gtk.cc?r1=11628&r2=11627
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_window_gtk.h?r1=11628&r2=11627

Fix some browser shutdown issues on Linux.

The previous code ended up destroying the BrowserWindow (removing it from  
BrowserList) while BrowserList was in the middle of iterating.  Push the  
deletion onto the message loop (using DeleteSoon) to emulate what I'm  
guessing happens on Windows (the message is queue instead of processed  
right away).

This fixes a second bug involving re-entrancy issues with Close() and  
window_, we now make sure to set window_ to NULL before any events might  
fire.

BUG=8712

Review URL: http://codereview.chromium.org/43135

------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to