Support sequence<T> as return type.

Support sequence<T> as return type.

R=podivilov@chromium.org

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

[podivilov, 2012-09-14 13:24:46]

http://codereview.chromium.org/10911307/diff/1/lib/html/scripts/generator.py
File lib/html/scripts/generator.py (right):

http://codereview.chromium.org/10911307/diff/1/lib/html/scripts/generator.py#...
lib/html/scripts/generator.py:674: def as_item_converter(self):
Why do you return primitive types from a function that is supposed to return
some kind of converter?

http://codereview.chromium.org/10911307/diff/1/lib/html/scripts/generator.py#...
lib/html/scripts/generator.py:677: if self.idl_type() == 'float': return 'float'
I don't think it's the preferred style in generator scripts. Please always
insert a new line after a colon.

[Anton Muhin, 2012-09-14 13:26:12]

http://codereview.chromium.org/10911307/diff/1/lib/html/scripts/generator.py
File lib/html/scripts/generator.py (right):

http://codereview.chromium.org/10911307/diff/1/lib/html/scripts/generator.py#...
lib/html/scripts/generator.py:674: def as_item_converter(self):
On 2012/09/14 13:24:46, podivilov wrote:
> Why do you return primitive types from a function that is supposed to return
> some kind of converter? 

The name is misleading, all I want is to return a generic argument to pass into
template.

Maybe later we'll return some traits, but I rather do this resolution in C++,
not generator

http://codereview.chromium.org/10911307/diff/1/lib/html/scripts/generator.py#...
lib/html/scripts/generator.py:677: if self.idl_type() == 'float': return 'float'
Will do

On 2012/09/14 13:24:46, podivilov wrote:
> I don't think it's the preferred style in generator scripts. Please always
> insert a new line after a colon.

[podivilov, 2012-09-14 15:15:03]

lgtm

http://codereview.chromium.org/10911307/diff/1/lib/html/scripts/generator.py
File lib/html/scripts/generator.py (right):

http://codereview.chromium.org/10911307/diff/1/lib/html/scripts/generator.py#...
lib/html/scripts/generator.py:644: def as_item_converter(self):
I hope it's temporary anyway, so let's call it
vector_to_dart_template_parameter. At least it's honest :)

[Anton Muhin, 2012-09-14 15:30:45]

http://codereview.chromium.org/10911307/diff/1/lib/html/scripts/generator.py
File lib/html/scripts/generator.py (right):

http://codereview.chromium.org/10911307/diff/1/lib/html/scripts/generator.py#...
lib/html/scripts/generator.py:644: def as_item_converter(self):
On 2012/09/14 15:15:04, podivilov wrote:
> I hope it's temporary anyway, so let's call it
> vector_to_dart_template_parameter. At least it's honest :)

Thanks a lot!