Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(58)

Side by Side Diff: pkg/dev_compiler/test/browser/language_tests.js

Issue 2789663005: Fix type checks and display for JS interop types. (Closed)
Patch Set: Code review comment fixes. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 define(['dart_sdk', 'async_helper', 'expect', 'unittest', 'is', 'require'], 5 define(['dart_sdk', 'async_helper', 'expect', 'unittest', 'is', 'require'],
6 function(dart_sdk, async_helper, expect, unittest, is, require) { 6 function(dart_sdk, async_helper, expect, unittest, is, require) {
7 'use strict'; 7 'use strict';
8 8
9 async_helper = async_helper.async_helper; 9 async_helper = async_helper.async_helper;
10 let minitest = expect.minitest; 10 let minitest = expect.minitest;
(...skipping 399 matching lines...) Expand 10 before | Expand all | Expand 10 after
410 410
411 // This is failing with a range error, I'm guessing because it's looking 411 // This is failing with a range error, I'm guessing because it's looking
412 // for a stylesheet and the page has none. 412 // for a stylesheet and the page has none.
413 'css_rule_list_test': 'fail', 413 'css_rule_list_test': 'fail',
414 414
415 'custom_element_method_clash_test': async_unittest, 415 'custom_element_method_clash_test': async_unittest,
416 'custom_element_name_clash_test': async_unittest, 416 'custom_element_name_clash_test': async_unittest,
417 'custom_elements_23127_test': async_unittest, 417 'custom_elements_23127_test': async_unittest,
418 'custom_elements_test': async_unittest, 418 'custom_elements_test': async_unittest,
419 419
420 // TODO(jmesserly): investigate the change here; it is likely due to 420 // Please do not mark this test as fail. If your change breaks this test,
421 // different reified types affecting the (gigantic) HTML literal 421 // please look at the test for instructions on how to generate a new
422 'debugger_test': fail, // firefox_fail 422 // golden file.
423 'debugger_test': firefox_fail,
423 'element_animate_test': 'unittest', 424 'element_animate_test': 'unittest',
424 425
425 // https://github.com/dart-lang/sdk/issues/27579. 426 // https://github.com/dart-lang/sdk/issues/27579.
426 'element_classes_test': 'fail', 427 'element_classes_test': 'fail',
427 'element_classes_svg_test': 'fail', 428 'element_classes_svg_test': 'fail',
428 429
429 // Failure: 'Expected 56 to be in the inclusive range [111, 160].'. 430 // Failure: 'Expected 56 to be in the inclusive range [111, 160].'.
430 'element_offset_test': 'fail', 431 'element_offset_test': 'fail',
431 432
432 'element_test': async_unittest, 433 'element_test': async_unittest,
(...skipping 481 matching lines...) Expand 10 before | Expand all | Expand 10 after
914 915
915 for (let action of unittest_tests) { 916 for (let action of unittest_tests) {
916 try { 917 try {
917 action(); 918 action();
918 } catch (e) { 919 } catch (e) {
919 console.error("Caught error tying to setup test:", e); 920 console.error("Caught error tying to setup test:", e);
920 } 921 }
921 } 922 }
922 }); 923 });
923 }); 924 });
OLDNEW
« no previous file with comments | « pkg/dev_compiler/lib/sdk/ddc_sdk.sum ('k') | pkg/dev_compiler/tool/input_sdk/private/ddc_runtime/types.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698