Make ClassNode visit type parameters in visitChildren

lib/compiler/implementation/tree/nodes.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 interface Visitor<R> {
   R visitBlock(Block node);
   R visitBreakStatement(BreakStatement node);
   R visitCascade(Cascade node);
   R visitCascadeReceiver(CascadeReceiver node);
   R visitCaseMatch(CaseMatch node);
@@ skipping 188 matching lines @@
             this.endToken);
 
   ClassNode asClassNode() => this;
 
   accept(Visitor visitor) => visitor.visitClassNode(this);
 
   visitChildren(Visitor visitor) {
     if (name !== null) name.accept(visitor);
     if (superclass !== null) superclass.accept(visitor);
     if (interfaces !== null) interfaces.accept(visitor);
+    if (typeParameters !== null) typeParameters.accept(visitor);

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

could there be any reason we didn't visit type parameters?

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

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

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

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:18]

207 in my CL. Done.

   }
 
   bool get isInterface() => beginToken.stringValue === 'interface';
 
   bool get isClass() => !isInterface;
 
   Token getBeginToken() => beginToken;
 
   Token getEndToken() => endToken;
 }
@@ skipping 1534 matching lines @@
  * argument).
  *
  * TODO(ahe): This method is controversial, the team needs to discuss
  * if top-level methods are acceptable and what naming conventions to
  * use.
  */
 initializerDo(Node node, f(Node node)) {
   SendSet send = node.asSendSet();
   if (send !== null) return f(send.arguments.head);
 }