Chromium Code Reviews
DescriptionRefactor partial interface resolving and merging into InterfaceMerger.pm
This is a more substantial refactoring than SemanticAnalyzer.pm
http://crrev.com/15984007
as it takes two sections of the body of generate-bindings.pl
and puts them into functions.
This makes the body of generate-bindings.pl very short (30 lines + parsing and headers), which makes it easier both:
1. to output/input JSON, since flow easier, and
2. to port to Python, since easier to review
This also removes “supplemental” from “supplemental dependencies”, as “supplemental” has technical meaning and, further, is redundant.
In Web IDL, “supplemental” is only used in the phrase “supplemental interface”, which uses the “implements” keyword. We're using it for partial interfaces, which is misleading, and we don't process “implements” (we parse and discard). Thus it's clearer to remove it and just call it “dependencies”.
Reference:
http://www.w3.org/TR/WebIDL/#dfn-supplemental-interface
supplemental interface
BUG=239771
Patch Set 1 #Patch Set 2 : Handle empties #
Messages
Total messages: 2 (0 generated)
|
|||||||||||||||||||||||||||||||||||||