Remove support for [EventTarget] IDL extended attribute

Remove support for [EventTarget] IDL extended attribute

Remove support for [EventTarget] IDL extended attribute now that all
interfaces inherit EventTarget instead.

Also remove EventTarget special casing from the bindings generator
as these are no longer needed.

BUG=257583
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=154010

[do-not-use, 2013-07-11 11:45:31]

https://codereview.chromium.org/18360008/diff/3001/Source/bindings/scripts/de...
File Source/bindings/scripts/deprecated_code_generator_v8.pm (left):

https://codereview.chromium.org/18360008/diff/3001/Source/bindings/scripts/de...
Source/bindings/scripts/deprecated_code_generator_v8.pm:5492: if
(IsDOMNodeType($type) || $type eq "EventTarget") {
This if case is not needed because it does exactly the same thing as the code
below (line 5514+). The only impact is potentially useless wtf header include.

[haraken, 2013-07-11 11:49:15]

LGTM. Great.

https://codereview.chromium.org/18360008/diff/3001/Source/bindings/scripts/de...
File Source/bindings/scripts/deprecated_code_generator_v8.pm (left):

https://codereview.chromium.org/18360008/diff/3001/Source/bindings/scripts/de...
Source/bindings/scripts/deprecated_code_generator_v8.pm:172: "EventTarget" => 1,

Are you sure that we can remove this? Last time I tried to remove this, I hit
some layout test failures. Maybe you have already fixed the issue in the process
of removing [EventTarget].

[do-not-use, 2013-07-11 12:00:33]

https://codereview.chromium.org/18360008/diff/3001/Source/bindings/scripts/de...
File Source/bindings/scripts/deprecated_code_generator_v8.pm (left):

https://codereview.chromium.org/18360008/diff/3001/Source/bindings/scripts/de...
Source/bindings/scripts/deprecated_code_generator_v8.pm:172: "EventTarget" => 1,
On 2013/07/11 11:49:16, haraken wrote:
> 
> Are you sure that we can remove this? Last time I tried to remove this, I hit
> some layout test failures. Maybe you have already fixed the issue in the
process
> of removing [EventTarget].

As you can see from the try bots, it does not cause failures. It does cause a
small diff in the generated bindings in 2 instances:
http://pastebin.com/zRKaMynQ

Based on the prototype of these 2 methods, the new code seems logical to me but
I may be wrong.

[haraken, 2013-07-11 12:05:07]

> As you can see from the try bots, it does not cause failures. It does cause a
> small diff in the generated bindings in 2 instances:
> http://pastebin.com/zRKaMynQ

Then it looks fine. Last time I tried, fast/dom/something failed in try bots.

[commit-bot: I haz the power, 2013-07-11 12:09:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@sisa.samsung.com/18360008/3001

[commit-bot: I haz the power, 2013-07-11 13:55:03]

Retried try job too often on win_layout_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...

[commit-bot: I haz the power, 2013-07-11 14:16:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@sisa.samsung.com/18360008/18001

[arv (Not doing code reviews), 2013-07-11 14:28:59]

LGTM

[commit-bot: I haz the power, 2013-07-11 15:24:46]

Change committed as 154010