[Webkit-unassigned] [Bug 188696] beforeunload interoperability issues with a throwing return

[bugzilla-daemon at webkit.org]

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

--- Comment #20 from PhistucK <phistuck at chromium.org> ---
@Darin and @Ryosuke - I agree that more tests are better than less tests.
I guess my fear of inflating the test suite comes from a thread regarding a totally different discussion I read some time ago that (from my sifting) regarded this as a serious problem -
https://lists.webkit.org/pipermail/webkit-dev/2017-November/029752.html

(It was actually quite traumatic to read on that day, but perhaps I simply misunderstood...)

Next steps -
- Fix the double evaluation issue.
- Research uncaughtExceptionInEventHandler.
- Add/adapt with required tests -
 - return {toString() { throw 'e' }} does not prevent navigation.
 - return {toString() { ++foo; throw 'e' }} as well as return {toString() { ++foo; return 'e'}} does not increase foo to more than 1.

Let me know if I missed anything, or if anything else is needed.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20180824/31cefb1f/attachment.html>