Explicitly mark first 2 args of addEventListener/removeEventListener as optional

Explicitly mark first 2 args of addEventListener/removeEventListener as optional

addEventListener and removeEventListener should require 2 arguments
(per spec), but legacy content calls these with 0 or 1 arguments.

For background, see this bug:
https://code.google.com/p/chromium/issues/detail?id=249598

This results in a special case in the CG, and an IDL that doesn't reflect
reality.

This updates the IDL to reflect reality, and removes the special-casing
from the CG, removing 3 FIXMEs!

There's now only 1 EventTarget FIXME in the templates, namely
the frame security check (on window).

This changes
addEventListener.length
removeEventListener.length
...from 2 to 0, reflecting reality.

R=haraken
BUG=345503
BUG=249598
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169631

[Nils Barth (inactive), 2014-03-19 03:26:21]

Further clean-up: EventTarget now almost normal!!

[haraken, 2014-03-19 03:34:19]

LGTM. Thanks for the tests.

[Nils Barth (inactive), 2014-03-19 03:35:05]

On 2014/03/19 03:34:19, haraken wrote:
> LGTM. Thanks for the tests.

N/p -- adding tests caught a crasher, so we shouldn't be hearing from
ClusterFuzz ;)

[Nils Barth (inactive), 2014-03-20 03:48:03]

The CQ bit was checked by nbarth@chromium.org

[commit-bot: I haz the power, 2014-03-20 03:48:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/203603005/80001

[commit-bot: I haz the power, 2014-03-20 04:27:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-20 04:27:13]

Try jobs failed on following builders:
  tryserver.blink on mac_blink_rel

[Nils Barth (inactive), 2014-03-20 06:01:06]

Updated test results (general length test was testing addEventListener),
so now returns actual value (0, not 2).

[Nils Barth (inactive), 2014-03-20 06:01:11]

The CQ bit was checked by nbarth@chromium.org

[commit-bot: I haz the power, 2014-03-20 06:01:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/203603005/100001

[commit-bot: I haz the power, 2014-03-20 07:09:14]

Change committed as 169631