IDL compiler: EventHandler attributes in Node interfaces

IDL compiler: EventHandler attributes in Node interfaces

Add test file for Node interfaces.
Also simplification of the hard-coded DOM Node list;
I should be able to eliminate this in Python thanks to interfaces_info
(in followup).

BUG=239771
R=haraken

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

[haraken, 2014-01-09 07:44:59]

LGTM.

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/scripts/...
File Source/bindings/scripts/code_generator_v8.pm (right):

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/scripts/...
Source/bindings/scripts/code_generator_v8.pm:205: my %domNodeTypes = ("Attr" =>
1,

This list is really fragile. Once we expose parsing trees of all interfaces,
this list will be gone :)

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/tests/id...
File Source/bindings/tests/idls/TestInterfaceInheritsNode.idl (right):

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/tests/id...
Source/bindings/tests/idls/TestInterfaceInheritsNode.idl:32: DependentLifetime, 
// FIXME: should inherit from Node

Why is this FIXME? Doesn't the inheritance work?

[Nils Barth (inactive), 2014-01-09 07:49:11]

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/scripts/...
File Source/bindings/scripts/code_generator_v8.pm (right):

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/scripts/...
Source/bindings/scripts/code_generator_v8.pm:205: my %domNodeTypes = ("Attr" =>
1,
On 2014/01/09 07:45:00, haraken wrote:
> This list is really fragile. Once we expose parsing trees of all interfaces,
> this list will be gone :)

I think interfaces_info should be enough (in Python)!
(I'll try it out in a followup.)

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/tests/id...
File Source/bindings/tests/idls/TestInterfaceInheritsNode.idl (right):

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/tests/id...
Source/bindings/tests/idls/TestInterfaceInheritsNode.idl:32: DependentLifetime, 
// FIXME: should inherit from Node
On 2014/01/09 07:45:00, haraken wrote:
> Why is this FIXME? Doesn't the inheritance work?

I haven't yet implemented inheriting Extended Attributes;
that was the main CL today, which I deferred.

(Currently we know the ancestor chain,
 but extended attributes on ancestors requires some Python
 parser/merge/reader changes, so I'll do that later,
 with [PutForwards].)

[haraken, 2014-01-09 07:50:37]

Still LGTM

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/scripts/...
File Source/bindings/scripts/code_generator_v8.pm (right):

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/scripts/...
Source/bindings/scripts/code_generator_v8.pm:205: my %domNodeTypes = ("Attr" =>
1,
On 2014/01/09 07:49:11, Nils Barth wrote:
> On 2014/01/09 07:45:00, haraken wrote:
> > This list is really fragile. Once we expose parsing trees of all interfaces,
> > this list will be gone :)
> 
> I think interfaces_info should be enough (in Python)!
> (I'll try it out in a followup.)

That sounds great.

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/tests/id...
File Source/bindings/tests/idls/TestInterfaceInheritsNode.idl (right):

https://codereview.chromium.org/131013002/diff/30001/Source/bindings/tests/id...
Source/bindings/tests/idls/TestInterfaceInheritsNode.idl:32: DependentLifetime, 
// FIXME: should inherit from Node
On 2014/01/09 07:49:11, Nils Barth wrote:
> On 2014/01/09 07:45:00, haraken wrote:
> > Why is this FIXME? Doesn't the inheritance work?
> 
> I haven't yet implemented inheriting Extended Attributes;
> that was the main CL today, which I deferred.
> 
> (Currently we know the ancestor chain,
>  but extended attributes on ancestors requires some Python
>  parser/merge/reader changes, so I'll do that later,
>  with [PutForwards].)

Makes sense. Thanks for the clarification.

[commit-bot: I haz the power, 2014-01-09 08:26:14]

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

[commit-bot: I haz the power, 2014-01-09 10:46:36]

Change committed as 164761