Introduce CompositeIDLTypeInfo and clean up the code that deals with sequences.

Introduce CompositeIDLTypeInfo and clean up the code that deals with sequences.

R=antonm@chromium.org

Committed: https://code.google.com/p/dart/source/detail?r=6735

[Anton Muhin, 2012-04-18 14:25:53]

https://chromiumcodereview.appspot.com/10116025/diff/3001/lib/dom/scripts/gen...
File lib/dom/scripts/generator.py (right):

https://chromiumcodereview.appspot.com/10116025/diff/3001/lib/dom/scripts/gen...
lib/dom/scripts/generator.py:444: if self._dart_type:
return self._dart_type or self._idl_type or it's too esoteric?

https://chromiumcodereview.appspot.com/10116025/diff/3001/lib/dom/scripts/gen...
lib/dom/scripts/generator.py:501: [self.idl_type()] + self._conversion_includes]
maybe always add self.idl_type() into self._conversion_includes?

https://chromiumcodereview.appspot.com/10116025/diff/3001/lib/dom/scripts/gen...
lib/dom/scripts/generator.py:512: class CompositeIDLTypeInfo(IDLTypeInfo):
do you want be that generic?  do we expect to have more than a list as a
collection?

[podivilov, 2012-04-18 14:56:26]

PTAL.

http://codereview.chromium.org/10116025/diff/3001/lib/dom/scripts/generator.py
File lib/dom/scripts/generator.py (right):

http://codereview.chromium.org/10116025/diff/3001/lib/dom/scripts/generator.p...
lib/dom/scripts/generator.py:444: if self._dart_type:
On 2012/04/18 14:25:53, antonmuhin wrote:
> return self._dart_type or self._idl_type or it's too esoteric?

Done.

http://codereview.chromium.org/10116025/diff/3001/lib/dom/scripts/generator.p...
lib/dom/scripts/generator.py:501: [self.idl_type()] + self._conversion_includes]
On 2012/04/18 14:25:53, antonmuhin wrote:
> maybe always add self.idl_type() into self._conversion_includes?

Done.

http://codereview.chromium.org/10116025/diff/3001/lib/dom/scripts/generator.p...
lib/dom/scripts/generator.py:512: class CompositeIDLTypeInfo(IDLTypeInfo):
On 2012/04/18 14:25:53, antonmuhin wrote:
> do you want be that generic?  do we expect to have more than a list as a
> collection?

Done.

[Anton Muhin, 2012-04-18 15:03:24]

LGTM!