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

Side by Side Diff: tools/testing/dart/compiler_configuration.dart

Issue 2685303002: Significantly reduce the timeout app_jit is given on the buildbot (Closed)
Patch Set: Created 3 years, 10 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
« no previous file with comments | « no previous file | tools/testing/dart/test_runner.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, 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 library compiler_configuration; 5 library compiler_configuration;
6 6
7 import 'dart:io' show Platform; 7 import 'dart:io' show Platform;
8 8
9 import 'runtime_configuration.dart' show RuntimeConfiguration; 9 import 'runtime_configuration.dart' show RuntimeConfiguration;
10 import 'runtime_configuration.dart' show DartPrecompiledAdbRuntimeConfiguration; 10 import 'runtime_configuration.dart' show DartPrecompiledAdbRuntimeConfiguration;
(...skipping 771 matching lines...) Expand 10 before | Expand all | Expand 10 after
782 ..addAll(sharedOptions) 782 ..addAll(sharedOptions)
783 ..addAll(originalArguments); 783 ..addAll(originalArguments);
784 } 784 }
785 } 785 }
786 786
787 class Dart2AppSnapshotCompilerConfiguration extends CompilerConfiguration { 787 class Dart2AppSnapshotCompilerConfiguration extends CompilerConfiguration {
788 Dart2AppSnapshotCompilerConfiguration({bool isDebug, bool isChecked}) 788 Dart2AppSnapshotCompilerConfiguration({bool isDebug, bool isChecked})
789 : super._subclass(isDebug: isDebug, isChecked: isChecked); 789 : super._subclass(isDebug: isDebug, isChecked: isChecked);
790 790
791 int computeTimeoutMultiplier() { 791 int computeTimeoutMultiplier() {
792 int multiplier = 2; 792 int multiplier = 1;
793 if (isDebug) multiplier *= 4; 793 if (isDebug) multiplier *= 2;
794 if (isChecked) multiplier *= 2; 794 if (isChecked) multiplier *= 2;
795 return multiplier; 795 return multiplier;
kustermann 2017/02/10 11:43:45 The returned value is applied again with 2 due to:
796 } 796 }
797 797
798 CommandArtifact computeCompilationArtifact( 798 CommandArtifact computeCompilationArtifact(
799 String buildDir, 799 String buildDir,
800 String tempDir, 800 String tempDir,
801 CommandBuilder commandBuilder, 801 CommandBuilder commandBuilder,
802 List arguments, 802 List arguments,
803 Map<String, String> environmentOverrides) { 803 Map<String, String> environmentOverrides) {
804 var snapshot = "$tempDir/out.jitsnapshot"; 804 var snapshot = "$tempDir/out.jitsnapshot";
805 return new CommandArtifact(<Command>[ 805 return new CommandArtifact(<Command>[
(...skipping 122 matching lines...) Expand 10 before | Expand all | Expand 10 after
928 RuntimeConfiguration runtimeConfiguration, 928 RuntimeConfiguration runtimeConfiguration,
929 String buildDir, 929 String buildDir,
930 TestInformation info, 930 TestInformation info,
931 List<String> vmOptions, 931 List<String> vmOptions,
932 List<String> sharedOptions, 932 List<String> sharedOptions,
933 List<String> originalArguments, 933 List<String> originalArguments,
934 CommandArtifact artifact) { 934 CommandArtifact artifact) {
935 return <String>[]; 935 return <String>[];
936 } 936 }
937 } 937 }
OLDNEW
« no previous file with comments | « no previous file | tools/testing/dart/test_runner.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698