Do not require upper bounds to be resolved and finalized before comparing them

runtime/vm/parser.cc

 // 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.
 
 #include "vm/parser.h"
 
 #include "vm/bigint_operations.h"
 #include "vm/class_finalizer.h"
 #include "vm/compiler.h"
 #include "vm/compiler_stats.h"
@@ skipping 2892 matching lines @@
                    factory_name.ident_pos));
     factory_class.set_library(library_);
     factory_class.set_is_finalized();
     ParseTypeParameters(factory_class);
     unresolved_factory_class.set_factory_signature_class(factory_class);
     interface.set_factory_class(unresolved_factory_class);
     // If a type parameter list is included in the default factory clause (it
     // can be omitted), verify that it matches the list of type parameters of
     // the interface in number and names.
     if (factory_class.NumTypeParameters() > 0) {
-      const TypeArguments& interface_type_parameters =
-          TypeArguments::Handle(interface.type_parameters());
-      const TypeArguments& factory_type_parameters =
-          TypeArguments::Handle(factory_class.type_parameters());
-      if (!TypeArguments::AreIdenticalTypeParameters(interface_type_parameters,
-                                                     factory_type_parameters)) {
+      if (!AbstractTypeArguments::AreIdentical(
+          AbstractTypeArguments::Handle(interface.type_parameters()),
+          AbstractTypeArguments::Handle(factory_class.type_parameters()))) {
         const String& interface_name = String::Handle(interface.Name());
         ErrorMsg(factory_name.ident_pos,
                  "mismatch in number or names of type parameters between "
                  "interface '%s' and default factory class '%s'.\n",
                  interface_name.ToCString(),
                  factory_name.ident->ToCString());
       }
     }
   }
 
@@ skipping 5352 matching lines @@
 void Parser::SkipQualIdent() {
   ASSERT(IsIdentifier());
   ConsumeToken();
   if (CurrentToken() == Token::kPERIOD) {
     ConsumeToken();  // Consume the kPERIOD token.
     ExpectIdentifier("identifier expected after '.'");
   }
 }
 
 }  // namespace dart