mirror of
https://github.com/mozilla/gecko-dev.git
synced 2025-01-22 12:04:38 +00:00
Had to temporarily back out danm's Release() hack from v1.95 as it causes a crash trying to load a new URL. While the hack did fix the problem with not being able to close the bookmark window my feeling is the crash is the less desirable behavior.
This commit is contained in:
parent
37edfc48db
commit
532becc3d2
@ -209,7 +209,12 @@ NS_IMETHODIMP nsWindow::Destroy()
|
||||
doing something really awful because it allows you to actually close (or appear to close)
|
||||
a window on the Mac, whose OS doesn't kill the window for you.
|
||||
*/
|
||||
Release(); // the ref added when nsWebShellWindow made us
|
||||
/* Unfortunately this hack has the nasty side effect of causing a crash when trying
|
||||
to load a new URL in either appRunner or viewer. Feeling that it is better to have
|
||||
windows that you can't close rather than to crash trying to browse I'm commenting
|
||||
out the Release() for now.
|
||||
*/
|
||||
//Release(); // the ref added when nsWebShellWindow made us
|
||||
|
||||
return NS_OK;
|
||||
}
|
||||
|
Loading…
x
Reference in New Issue
Block a user