Fix type-checks for reflective redirection constructors.

tests/lib/lib.status

 # 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.
 
 [ $checked && $compiler != dartanalyzer && $compiler != dart2analyzer && $runtime != dartium ]
 mirrors/regress_16321_test/01: CompileTimeError # Issue 16351
 
 [ $checked && $compiler == none && ($runtime == dartium || $runtime == drt) ]
 mirrors/regress_16321_test/01: RunTimeError # Issue 16351
 
 [ $compiler == dart2js && $checked && $runtime == ie9 ]
 convert/ascii_test: Skip # http://dartbug.com/15498
 convert/streamed_conversion_json_utf8_encode_test: Skip # http://dartbug.com/15498
 convert/streamed_conversion_utf8_encode_test: Skip # http://dartbug.com/15498
 
 mirrors/mirrors_reader_test: Skip # Issue 16589
 
+[ $unchecked ]
+mirrors/redirecting_factory_different_type_test: SkipByDesign # Tests type checks.
+
 [ $csp ]
 async/deferred/deferred_in_isolate_test: RuntimeError # Issue 16898
 mirrors/deferred_mirrors_metadata_test: Fail # Issue 16898
 mirrors/deferred_mirrors_metatarget_test: Fail # Issue 16898
 
 [ $compiler == dart2js ]
 async/schedule_microtask6_test: RuntimeError # global error handling is not supported. http://dartbug.com/5958
 
 math/double_pow_test: RuntimeError
 math/low_test: RuntimeError
@@ skipping 265 matching lines @@
 mirrors/redirecting_factory_test/01: StaticWarning # test issue X, The return type 'Class<T2, T1>' of the redirected constructor is not assignable to 'Class<T1, T2>'
 mirrors/redirecting_factory_test/none: StaticWarning # test issue X, The return type 'Class<T2, T1>' of the redirected constructor is not assignable to 'Class<T1, T2>
 
 mirrors/immutable_collections_test: StaticWarning, OK # Expect failure for any type of Iterable.
 mirrors/inference_and_no_such_method_test: StaticWarning, OK # Expect to trigger noSuchMethod.
 mirrors/mirrors_nsm_test: StaticWarning, OK # Expect to trigger noSuchMethod.
 
 mirrors/repeated_private_anon_mixin_app_test: StaticWarning, OK # Intentional library name conflict.
 mirrors/removed_api_test: StaticWarning, OK # Deliberately refers to undeclared members.
 
+mirrors/redirecting_factory_different_type_test: StaticWarning, OK # By design.
+
 mirrors/deferred_mirrors_metadata_test: Fail # Issue 17522
 
 [ $compiler == dart2js && $runtime == d8 && $system == windows ]
 async/*deferred*: Skip # Issue 17458
 mirrors/*deferred*: Skip # Issue 17458
 
 [ $compiler == dart2js && $mode == debug ]
 mirrors/native_class_test: Pass, Slow
 
 [ $arch == simmips || $arch == mips ]
 convert/chunked_conversion_utf88_test: Skip  # Pass, Slow Issue 12025.
 convert/streamed_conversion_json_utf8_decode_test: Skip  # Pass, Slow
 
 [ $arch == simarm ]
 convert/chunked_conversion_utf88_test: Skip  # Pass, Slow Issue 12644.
 convert/utf85_test: Skip  # Pass, Slow Issue 12644.
 
 [ $arch == simarm64 ]
 convert/utf85_test: Skip # Pass, Slow Issue 20111.
 
 # These tests use "const Symbol('void')".
 [ $compiler == dart2analyzer ]
 mirrors/basic_types_in_dart_core_test: CompileTimeError
 mirrors/function_type_mirror_test: CompileTimeError
 mirrors/method_mirror_returntype_test: CompileTimeError