fix #569, cache constants defined in methods

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 451 matching lines @@
       // 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': fail,
       'encoding_test': skip_timeout,
 
+      // TODO(jmesserly): this is in an inconsistent state between our old and
+      // newer SDKs.
+      'html_escape_test': skip_fail,

[sra1, 2016/05/14 19:58:27]

maybe just fail so we know when it is fixed.

[Jennifer Messerly, 2016/05/17 18:04:14]

The problem here is it's failing inconsistently. Using an old SDK it will pass
(e.g. Travis bot). If using a new SDK, it will fail. Hence skip_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': fail,
 
       'streamed_conversion_json_encode1_test': skip_timeout,
@@ skipping 330 matching lines @@
 
     for (let action of unittest_tests) {
       try {
         action();
       } catch (e) {
         console.error("Caught error tying to setup test:", e);
       }
     }
   });
 })();