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

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: Fix type checks and display for JS interop types. 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 396 matching lines...) Expand 10 before | Expand all | Expand 10 after
407 407
408 // This is failing with a range error, I'm guessing because it's looking 408 // This is failing with a range error, I'm guessing because it's looking
409 // for a stylesheet and the page has none. 409 // for a stylesheet and the page has none.
410 'css_rule_list_test': 'fail', 410 'css_rule_list_test': 'fail',
411 411
412 'custom_element_method_clash_test': async_unittest, 412 'custom_element_method_clash_test': async_unittest,
413 'custom_element_name_clash_test': async_unittest, 413 'custom_element_name_clash_test': async_unittest,
414 'custom_elements_23127_test': async_unittest, 414 'custom_elements_23127_test': async_unittest,
415 'custom_elements_test': async_unittest, 415 'custom_elements_test': async_unittest,
416 416
417 // TODO(jmesserly): investigate the change here; it is likely due to 417 // Please do not mark this test as fail. If your change breaks this test,
418 // different reified types affecting the (gigantic) HTML literal 418 // copy and paste the gigantic JSON literal shown in the test output
419 'debugger_test': fail, // firefox_fail 419 // into debugger_test.dart and then view the diff with your favorite diff
420 // viewing tool to make sure it looks reasonable. Generally it should be
421 // fairly obvious if the changes are reasonable or not.
422 'debugger_test': firefox_fail,
420 'element_animate_test': 'unittest', 423 'element_animate_test': 'unittest',
421 424
422 // https://github.com/dart-lang/sdk/issues/27579. 425 // https://github.com/dart-lang/sdk/issues/27579.
423 'element_classes_test': 'fail', 426 'element_classes_test': 'fail',
424 'element_classes_svg_test': 'fail', 427 'element_classes_svg_test': 'fail',
425 428
426 // Failure: 'Expected 56 to be in the inclusive range [111, 160].'. 429 // Failure: 'Expected 56 to be in the inclusive range [111, 160].'.
427 'element_offset_test': 'fail', 430 'element_offset_test': 'fail',
428 431
429 'element_test': async_unittest, 432 'element_test': async_unittest,
(...skipping 479 matching lines...) Expand 10 before | Expand all | Expand 10 after
909 912
910 for (let action of unittest_tests) { 913 for (let action of unittest_tests) {
911 try { 914 try {
912 action(); 915 action();
913 } catch (e) { 916 } catch (e) {
914 console.error("Caught error tying to setup test:", e); 917 console.error("Caught error tying to setup test:", e);
915 } 918 }
916 } 919 }
917 }); 920 });
918 }); 921 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698