[Webkit-unassigned] [Bug 27481] onbeforeunload not called at window close + frame or iframe focused

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=27481


Darin Adler <darin at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #48156|review?                     |review-
               Flag|                            |




--- Comment #5 from Darin Adler <darin at apple.com>  2010-02-05 11:37:44 PST ---
(From update of attachment 48156)
This looks like the correct fix. It is good to make Windows and Chromium match
the Mac in this respect.

> +        Layout tests aren't able to test this bug, since it requires closing
> +        the actual browser window, not calling window.close().

Did you try to make a test?

I understand that window.close() is not helpful to test this, but it should be
testable in DumpRenderTree. We have existing tests for beforeunload and they
had no problem with this. For example, onunload-clears-onbeforeunload.html.

If you do need a way to test this, we would typically add a way to do it to
DumpRenderTree. We have features already to test opening and closing multiple
windows. You can find 73 tests that use the
layoutTestController.setCanOpenWindows() and you can probably create a test
that way.

If you try all these things and fail, at least you should add a test to the
WebCore/manual-tests directory.

review- because this does need a test, and further, an automated test should be
possible

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.