65802 2011-08-05 17:45:52 -0700 Add tests for whether arguments are optional for addEventListener and removeEventListener 2011-10-07 10:22:12 -0700 1 1 1 Unclassified WebKit WebCore JavaScript 528+ (Nightly build) Unspecified Unspecified NEW P2 Normal --- 65787 1 abarth webkit-unassigned sam oldest_to_newest 447510 0 abarth 2011-08-05 17:45:52 -0700 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. 479750 1 abarth 2011-10-06 19:55:21 -0700 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. 480105 2 abarth 2011-10-07 10:21:47 -0700 I think the best resolution here is to add tests and to be on the lookout for compatibility problems.