Skip to content

Commit 744bb47

Browse files
athomascommit-bot@chromium.org
authored andcommitted
[infra] Remove custom timeouts for dart2js compiler config
These were probably added in Dart 1 because --checked carried a much higher performance penalty. Today, most tests finish within 1 minute. If certain configs need longer timeouts these should be specified in the named configuration in test_matrix.json. Change-Id: I91c456475971059297e16b9902dc16c2b23f9c9e Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/107880 Reviewed-by: William Hesse <whesse@google.com> Commit-Queue: Alexander Thomas <athom@google.com>
1 parent 9b53686 commit 744bb47

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

pkg/test_runner/lib/src/compiler_configuration.dart

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -419,14 +419,6 @@ class Dart2jsCompilerConfiguration extends Dart2xCompilerConfiguration {
419419
Dart2jsCompilerConfiguration(TestConfiguration configuration)
420420
: super('dart2js', configuration);
421421

422-
int get timeoutMultiplier {
423-
var multiplier = 1;
424-
if (_isDebug) multiplier *= 4;
425-
if (_isChecked) multiplier *= 2;
426-
if (_isHostChecked) multiplier *= 16;
427-
return multiplier;
428-
}
429-
430422
List<String> computeCompilerArguments(
431423
List<String> vmOptions,
432424
List<String> sharedOptions,

tests/co19_2/co19_2-dart2js.status

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,16 @@ LayoutTests/*: SkipByDesign # d8 is not a browser
1010
LibTest/html/*: SkipByDesign # d8 is not a browser
1111
WebPlatformTest/*: SkipByDesign # d8 is not a browser
1212

13+
[ $compiler == dart2js && $runtime == d8 && $host_checked ]
14+
LibTest/collection/ListBase/ListBase_class_A01_t04: Slow
15+
LibTest/collection/ListBase/ListBase_class_A01_t05: Slow
16+
LibTest/collection/ListBase/ListBase_class_A01_t06: Slow
17+
LibTest/collection/ListMixin/ListMixin_class_A01_t04: Slow
18+
LibTest/collection/ListMixin/ListMixin_class_A01_t05: Slow
19+
LibTest/collection/ListMixin/ListMixin_class_A01_t06: Slow
20+
LibTest/core/List/List_class_A01_t05: Slow
21+
LibTest/core/List/List_class_A01_t06: Slow
22+
1323
[ $compiler == dart2js && $runtime == ie11 ]
1424
LibTest/collection/ListBase/ListBase_class_A01_t04: SkipSlow # slow babeljs transformation
1525
LibTest/collection/ListBase/ListBase_class_A01_t05: SkipSlow # slow babeljs transformation

0 commit comments

Comments
 (0)