Throw TypeError instead of CastError for type coercions.

test/browser/language_tests.js

Index: test/browser/language_tests.js
diff --git a/test/browser/language_tests.js b/test/browser/language_tests.js
index 21c71e3af13dc3b9a5c04787257d1289ae5798d0..6f51b94de54a359f1621f74d7f1addf389b7327c 100644
--- a/test/browser/language_tests.js
+++ b/test/browser/language_tests.js
@@ -33,7 +33,6 @@
   // TODO(jmesserly): separate StrongModeError from other errors.
   let all_status = {
     'language': {
-      'arithmetic2_test': fail,
       'assert_with_type_test_or_cast_test': skip_fail,
       'assertion_test': skip_fail,
       'async_await_test_none_multi': 'unittest',
@@ -137,7 +136,7 @@
       'double_to_string_test': skip_fail,
       'dynamic_test': skip_fail,
       'enum_mirror_test': skip_fail,
-      'exception_test': skip_fail,
+      'exception_test': fail,
       'execute_finally6_test': skip_fail,
       'expect_test': skip_fail,
       'extends_test_lib': skip_fail,
@@ -149,15 +148,10 @@
       'fast_method_extraction_test': skip_fail,
       'field_increment_bailout_test': skip_fail,
       'field_optimization3_test': skip_fail,
-      'field_test': skip_fail,
       'final_syntax_test_08_multi': skip_fail,
-      'first_class_types_literals_test_01_multi': skip_fail,
-      'first_class_types_literals_test_02_multi': skip_fail,
-      'first_class_types_literals_test_none_multi': skip_fail,
       'first_class_types_test': skip_fail,
       'for_in2_test': skip_fail,
       'for_variable_capture_test': skip_fail,
-      'function_propagation_test': skip_fail,
       'function_subtype0_test': skip_fail,
       'function_subtype1_test': skip_fail,
       'function_subtype2_test': skip_fail,
@@ -205,38 +199,27 @@
       'function_type_alias_test': skip_fail,
       'function_type_call_getter_test': skip_fail,
       'gc_test': skip_fail,
-      'generic2_test': skip_fail,
-      'generic_deep_test': skip_fail,
       'generic_field_mixin2_test': skip_fail,
       'generic_field_mixin3_test': skip_fail,
       'generic_field_mixin4_test': skip_fail,
       'generic_field_mixin5_test': skip_fail,
       'generic_field_mixin_test': skip_fail,
-      'generic_inheritance_test': skip_fail,
       'generic_instanceof2_test': skip_fail,
-      'generic_instanceof3_test': skip_fail,
-      'generic_instanceof_test': skip_fail,
       'generic_is_check_test': skip_fail,
-      'generic_native_test': skip_fail,
-      'generic_parameterized_extends_test': skip_fail,
       'getter_closure_execution_order_test': skip_fail,
       'getter_override2_test_00_multi': skip_fail,
       'getters_setters_test': skip_fail,
       'hash_code_mangling_test': skip_fail,
       'identical_closure2_test': skip_fail,
-      'if_null_behavior_test_14_multi': skip_fail,
       'infinite_switch_label_test': skip_fail,
       'infinity_test': skip_fail,
       'instance_creation_in_function_annotation_test': skip_fail,
-      'instanceof2_test': skip_fail,
-      'instanceof4_test_01_multi': skip_fail,
-      'instanceof4_test_none_multi': skip_fail,
+      'instanceof2_test': fail,
+      'instanceof4_test_01_multi': fail,
+      'instanceof4_test_none_multi': fail,
       'instanceof_optimized_test': skip_fail,
-      'int_test': skip_fail,
-      'integer_division_by_zero_test': skip_fail,
-      'interceptor_test': skip_fail,
-      'interceptor9_test': skip_fail,
-      'is_nan_test': skip_fail,
+      'integer_division_by_zero_test': fail,
+      'is_nan_test': fail,
       'issue10747_test': skip_fail,
       'issue13179_test': skip_fail,
       'issue21079_test': skip_fail,
@@ -273,20 +256,16 @@
       'mixin_type_parameter1_test': skip_fail,
       'mixin_type_parameter2_test': skip_fail,
       'mixin_type_parameter3_test': skip_fail,
-      'modulo_test': skip_fail,
-      'named_argument_test': skip_fail,
+      'modulo_test': fail,
       'named_parameter_clash_test': skip_fail,
-      'namer2_test': skip_fail,
       'nan_identical_test': skip_fail,
       'nested_switch_label_test': skip_fail,
       'no_such_method3_test': skip_fail,
-      'no_such_method_empty_selector_test': skip_fail,
+      'no_such_method_empty_selector_test': fail,
       'no_such_method_subtype_test': skip_fail,
-      'null_no_such_method_test': skip_fail,
       'number_identifier_test_05_multi': skip_fail,
       'number_identity2_test': skip_fail,
       'numbers_test': skip_fail,
-      'operator4_test': skip_fail,  // CastError instead of TypeError
       'optimized_hoisting_checked_mode_assert_test': skip_fail,
       'redirecting_factory_reflection_test': skip_fail,
       'regress_13462_0_test': skip_fail,
@@ -297,11 +276,7 @@
       'regress_21795_test': skip_fail,
       'regress_22443_test': skip_fail,
       'regress_22666_test': skip_fail,
-      'regress_22719_test': skip_fail,
-      'regress_23650_test': skip_fail,
-      'regress_r24720_test': skip_fail,
       'setter_no_getter_test_01_multi': skip_fail,
-      'smi_type_test': skip_fail,
       'stack_overflow_stacktrace_test': skip_fail,
       'stack_overflow_test': skip_fail,
       'stack_trace_test': skip_fail,
@@ -310,7 +285,6 @@
       'stacktrace_rethrow_error_test_withtraceparameter_multi': skip_fail,
       'stacktrace_test': skip_fail,
       'string_interpolate_null_test': skip_fail,
-      'string_interpolation_newline_test': skip_fail,
       'super_field_2_test': skip_fail,
       'super_field_test': skip_fail,
       'super_operator_index3_test': skip_fail,
@@ -321,15 +295,9 @@
       'sync_generator1_test_none_multi': skip_fail,
       'throwing_lazy_variable_test': skip_fail,
       'top_level_non_prefixed_library_test': skip_fail,
-      'truncdiv_test': skip_fail,
-      'type_argument_substitution_test': skip_fail,
-      'type_promotion_functions_test_none_multi': skip_fail,
-      'type_variable_closure2_test': skip_fail,
-      'type_variable_field_initializer_closure_test': skip_fail,
-      'type_variable_field_initializer_test': skip_fail,
-      'type_variable_nested_test': skip_fail,
-      'type_variable_typedef_test': skip_fail,
-      'typedef_is_test': skip_fail,
+      'truncdiv_test': fail,  // did not throw
+      'type_variable_nested_test': skip_fail,  // unsound is-check
+      'type_variable_typedef_test': skip_fail,  // unsound is-check
 
       'bit_operations_test_01_multi': skip_fail,
       'bit_operations_test_02_multi': skip_fail,
@@ -361,16 +329,6 @@
       // https://github.com/dart-lang/sdk/issues/26124
       'prefix10_negative_test': skip_fail,
 
-      // TODO(vsm): Right shift should not propagate sign
-      // https://github.com/dart-lang/dev_compiler/issues/446
-      'float32x4_sign_mask_test': skip_fail,
-      'int32x4_sign_mask_test': skip_fail,
-
-      // TODO(vsm): Triage further
-      // exports._GeneratorIterable$ is not a function
-      'byte_data_test': skip_fail,
-      'endianness_test': skip_fail,
-
       'library_prefixes_test1': 'helper',
       'library_prefixes_test2': 'helper',
       'top_level_prefixed_library_test': 'helper',