Make sure the signature of the call method is finalized before involving it

runtime/lib/mirrors.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 "lib/mirrors.h"
 
 #include "lib/invocation_mirror.h"
 #include "vm/bootstrap_natives.h"
 #include "vm/class_finalizer.h"
 #include "vm/compiler.h"
@@ skipping 856 matching lines @@
                               "Type arguments must be instances of Type.")));
       Exceptions::ThrowByType(Exceptions::kArgumentValue, error_args);
       UNREACHABLE();
     }
     type_arg ^= args.At(i);
     type_args_obj.SetTypeAt(i, type_arg);
   }
 
   Type& instantiated_type =
       Type::Handle(Type::New(clz, type_args_obj, TokenPosition::kNoSource));
-  instantiated_type ^= ClassFinalizer::FinalizeType(
-      clz, instantiated_type, ClassFinalizer::kCanonicalize);
+  instantiated_type ^= ClassFinalizer::FinalizeType(clz, instantiated_type);
   if (instantiated_type.IsMalbounded()) {
     const LanguageError& type_error =
         LanguageError::Handle(instantiated_type.error());
     const Array& error_args = Array::Handle(Array::New(3));
     error_args.SetAt(0, args);
     error_args.SetAt(1, String::Handle(String::New("typeArguments")));
     error_args.SetAt(2, String::Handle(type_error.FormatMessage()));
     Exceptions::ThrowByType(Exceptions::kArgumentValue, error_args);
     UNREACHABLE();
   }
@@ skipping 1199 matching lines @@
 
 DEFINE_NATIVE_ENTRY(TypeMirror_subtypeTest, 2) {
   GET_NON_NULL_NATIVE_ARGUMENT(AbstractType, a, arguments->NativeArgAt(0));
   GET_NON_NULL_NATIVE_ARGUMENT(AbstractType, b, arguments->NativeArgAt(1));
   return Bool::Get(a.IsSubtypeOf(b, NULL, NULL, Heap::kNew)).raw();
 }
 
 #endif  // !PRODUCT && !DART_PRECOMPILED_RUNTIME
 
 }  // namespace dart