allow nsXULWindow::CreateNewWindow to fail; don't fall back on the appshell. bug 115969 r=jaggernaut,jst
git-svn-id: svn://10.0.0.236/trunk@111341 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
@@ -106,10 +106,9 @@ nsWindowCreator::CreateChromeWindow(nsIWebBrowserChrome *aParent,
|
||||
NS_ASSERTION(xulParent, "window created using non-XUL parent. that's unexpected, but may work.");
|
||||
if (xulParent)
|
||||
xulParent->CreateNewWindow(aChromeFlags, getter_AddRefs(newWindow));
|
||||
}
|
||||
|
||||
// no parent or incompetent parent. try again using basic methods:
|
||||
if (!newWindow) {
|
||||
// And if it fails, don't try again without a parent. It could fail
|
||||
// intentionally (bug 115969).
|
||||
} else { // try using basic methods:
|
||||
/* You really shouldn't be making dependent windows without a parent.
|
||||
But unparented modal (and therefore dependent) windows happen
|
||||
in our codebase, so we allow it after some bellyaching: */
|
||||
|
||||
Reference in New Issue
Block a user