NEW 65802
Add tests for whether arguments are optional for addEventListener and removeEventListener 
https://bugs.webkit.org/show_bug.cgi?id=65802
Summary Add tests for whether arguments are optional for addEventListener and removeE...
Adam Barth
Reported 2011-08-05 17:45:52 PDT
As described in https://bugs.webkit.org/show_bug.cgi?id=65771, the patch in bug https://bugs.webkit.org/show_bug.cgi?id=63417 combined with the legacy default argument removal seemed to affect the behavior of addEventListener/removeEventListener. I need to look in the code generator and understand why / we need to determine which behavior we want.
Attachments
Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2011-10-06 19:55:21 PDT
Although these are special-cased in the code generator, it appears that the argument handling uses some of normal argument handling code path, so we did change behavior here. Our new behavior is more aligned with Firefox, so I'm inclined to wait and see if we receive any compatibility complaints. This has been in the tree for a bit and I'm not aware of any compat issues. If you know of any, please let me know.
Adam Barth
Comment 2 2011-10-07 10:21:47 PDT
I think the best resolution here is to add tests and to be on the lookout for compatibility problems.
Note You need to log in before you can comment on or make changes to this bug.