Make ClassNode visit type parameters in visitChildren

Make ClassNode visit type parameters in visitChildren

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

[Roman, 2012-08-21 14:36:37]

https://chromiumcodereview.appspot.com/10831406/diff/1/lib/compiler/implement...
File lib/compiler/implementation/tree/nodes.dart (right):

https://chromiumcodereview.appspot.com/10831406/diff/1/lib/compiler/implement...
lib/compiler/implementation/tree/nodes.dart:209: if (typeParameters !== null)
typeParameters.accept(visitor);
could there be any reason we didn't visit type parameters?

[ahe, 2012-08-21 14:45:27]

The test is great, but please coordinate with messick.

https://chromiumcodereview.appspot.com/10831406/diff/1/lib/compiler/implement...
File lib/compiler/implementation/tree/nodes.dart (right):

https://chromiumcodereview.appspot.com/10831406/diff/1/lib/compiler/implement...
lib/compiler/implementation/tree/nodes.dart:209: if (typeParameters !== null)
typeParameters.accept(visitor);
On 2012/08/21 14:36:37, Roman wrote:
> could there be any reason we didn't visit type parameters?

Oversight. However, messick already have a CL in the works for this.

[Anton Muhin, 2012-08-21 14:46:26]

lgtm

[messick, 2012-08-21 14:56:37]

https://chromiumcodereview.appspot.com/10831406/diff/1/lib/compiler/implement...
File lib/compiler/implementation/tree/nodes.dart (right):

https://chromiumcodereview.appspot.com/10831406/diff/1/lib/compiler/implement...
lib/compiler/implementation/tree/nodes.dart:209: if (typeParameters !== null)
typeParameters.accept(visitor);
On 2012/08/21 14:45:27, ahe wrote:
> On 2012/08/21 14:36:37, Roman wrote:
> > could there be any reason we didn't visit type parameters?
> 
> Oversight. However, messick already have a CL in the works for this.

Please commit this CL after moving the new code to line 208 as Peter suggested
in my CL. (Odd that we would both put it in the same wrong place!)

[Roman, 2012-08-21 15:00:17]

https://chromiumcodereview.appspot.com/10831406/diff/1/lib/compiler/implement...
File lib/compiler/implementation/tree/nodes.dart (right):

https://chromiumcodereview.appspot.com/10831406/diff/1/lib/compiler/implement...
lib/compiler/implementation/tree/nodes.dart:209: if (typeParameters !== null)
typeParameters.accept(visitor);
On 2012/08/21 14:56:37, messick wrote:
> On 2012/08/21 14:45:27, ahe wrote:
> > On 2012/08/21 14:36:37, Roman wrote:
> > > could there be any reason we didn't visit type parameters?
> > 
> > Oversight. However, messick already have a CL in the works for this.
> 
> Please commit this CL after moving the new code to line 208 as Peter suggested
> in my CL. (Odd that we would both put it in the same wrong place!)

207 in my CL. Done.

[ahe, 2012-08-21 15:02:26]

LGTM!