Mark some tests that could be fail instead of skip_fail as fail. Mark timeout test as timeout inste…

test/browser/language_tests.js

 // Copyright (c) 2015, 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.
 
 (function() {
   'use strict';
 
   let dart_sdk = dart_library.import('dart_sdk');
   dart_sdk._isolate_helper.startRootIsolate(function() {}, []);
   let async_helper = dart_library.import('async_helper').async_helper;
@@ skipping 443 matching lines @@
       'throw_half_surrogate_pair_test_01_multi': fail,
       // TODO(rnystrom): Times out because it tests a huge number of
       // combinations of URLs (4 * 5 * 5 * 8 * 6 * 6 * 4 = 115200).
       'uri_parse_test': skip_timeout,
     },
 
     'lib/convert': {
       // TODO(jmesserly): this could be a failure we didn't notice before
       // because we now call the right Dart double.toString, resulting in a
       // difference between -0.0 and 0?
-      'chunked_conversion_json_encode1_test': skip_fail,
+      'chunked_conversion_json_encode1_test': fail,
 
       // TODO(rnystrom): A lot of the convert tests timeout. Some do pass if
       // you increase the time by a large amount, but it's pretty gratuitous.
       // I'm not sure why they are so slow. One guess is that they are spewing
       // a ton of warnings, that slow down the test.
       'chunked_conversion_utf84_test': skip_timeout,
       'chunked_conversion_utf88_test': skip_timeout,
       'chunked_conversion_utf8_test': skip_timeout,
 
       // TODO(rnystrom): Strong mode cast failures.
-      'codec1_test': skip_fail,
-      'encoding_test': skip_fail,
+      'codec1_test': fail,
+      'encoding_test': skip_timeout,
       'html_escape_test': fail,
 
       // TODO(rnystrom): If this test is enabled, karma gets confused and
       // disconnects randomly.
       'json_lib_test': skip_fail,
 
       'json_utf8_chunk_test': skip_timeout,
 
       // TODO(rnystrom): Strong mode cast failure.
-      'line_splitter_test': skip_fail,
+      'line_splitter_test': fail,
 
       'streamed_conversion_json_encode1_test': skip_timeout,
       'streamed_conversion_json_utf8_decode_test': skip_timeout,
       'streamed_conversion_json_utf8_encode_test': skip_timeout,
       'streamed_conversion_utf8_decode_test': skip_timeout,
       'streamed_conversion_utf8_encode_test': skip_timeout,
       'utf85_test': skip_timeout,
     },
 
     // TODO(jacobr): enable more of the html tests in unittest once they have
@@ skipping 322 matching lines @@
 
     for (let action of unittest_tests) {
       try {
         action();
       } catch (e) {
         console.error("Caught error tying to setup test:", e);
       }
     }
   });
 })();