Commit 48a4a372 authored by mjs's avatar mjs

Reviewed by John.

	- fixed 3149511 - Blogger.com window closes after update is posted

        * khtml/ecma/kjs_window.cpp:
        (WindowFunc::tryCall): For now, don't allow closing the window at
	all if it wasn't opened by JavaScript, since we don't support the
	history object and so can't tell if the window has only one
	history item.


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3355 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 0a04d8f2
2003-01-17 Maciej Stachowiak <mjs@apple.com>
Reviewed by John.
- fixed 3149511 - Blogger.com window closes after update is posted
* khtml/ecma/kjs_window.cpp:
(WindowFunc::tryCall): For now, don't allow closing the window at
all if it wasn't opened by JavaScript, since we don't support the
history object and so can't tell if the window has only one
history item.
2003-01-17 Darin Adler <darin@apple.com>
Reviewed by Dave.
......
2003-01-17 Maciej Stachowiak <mjs@apple.com>
Reviewed by John.
- fixed 3149511 - Blogger.com window closes after update is posted
* khtml/ecma/kjs_window.cpp:
(WindowFunc::tryCall): For now, don't allow closing the window at
all if it wasn't opened by JavaScript, since we don't support the
history object and so can't tell if the window has only one
history item.
2003-01-17 Darin Adler <darin@apple.com>
Reviewed by Dave.
......
......@@ -1391,7 +1391,13 @@ Value WindowFunc::tryCall(ExecState *exec, Object &thisObj, const List &args)
|| KMessageBox::questionYesNo( window->part()->widget(), i18n("Close window?"), i18n("Confirmation Required") ) == KMessageBox::Yes
#endif
)
// FIXME: we can't ever close a non-JS window until we fix the
// history object to really work.
#if APPLE_CHANGES
;
#else
(const_cast<Window*>(window))->scheduleClose();
#endif
}
else
{
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment