Skip to content

Commit 21b8b72

Browse files
[Impeller] Temporary flag flip for devicelab tests to use Impeller. (#122224)
[Impeller] Temporary flag flip for devicelab tests to use Impeller.
1 parent 2e7b458 commit 21b8b72

21 files changed

+41
-20
lines changed

dev/devicelab/bin/tasks/animated_complex_opacity_perf_impeller_ios__e2e_summary.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,5 +10,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
1010

1111
Future<void> main() async {
1212
deviceOperatingSystem = DeviceOperatingSystem.ios;
13+
// ignore: avoid_redundant_argument_values
1314
await task(createAnimatedComplexOpacityPerfE2ETest(enableImpeller: true));
1415
}

dev/devicelab/bin/tasks/backdrop_filter_perf_impeller_ios__timeline_summary.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
88

99
Future<void> main() async {
1010
deviceOperatingSystem = DeviceOperatingSystem.ios;
11+
// ignore: avoid_redundant_argument_values
1112
await task(createBackdropFilterPerfTest(enableImpeller: true));
1213
}

dev/devicelab/bin/tasks/color_filter_and_fade_perf_impeller_ios__e2e_summary.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,5 +10,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
1010

1111
Future<void> main() async {
1212
deviceOperatingSystem = DeviceOperatingSystem.ios;
13+
// ignore: avoid_redundant_argument_values
1314
await task(createColorFilterAndFadePerfE2ETest(enableImpeller: true));
1415
}

dev/devicelab/bin/tasks/complex_layout_scroll_perf_bad_impeller_ios__timeline_summary.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
88

99
Future<void> main() async {
1010
deviceOperatingSystem = DeviceOperatingSystem.ios;
11+
// ignore: avoid_redundant_argument_values
1112
await task(createComplexLayoutScrollPerfTest(badScroll: true, enableImpeller: true));
1213
}

dev/devicelab/bin/tasks/complex_layout_scroll_perf_impeller_ios__timeline_summary.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
88

99
Future<void> main() async {
1010
deviceOperatingSystem = DeviceOperatingSystem.ios;
11+
// ignore: avoid_redundant_argument_values
1112
await task(createComplexLayoutScrollPerfTest(enableImpeller: true));
1213
}

dev/devicelab/bin/tasks/flutter_gallery__transition_perf_e2e_impeller_ios.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@ import 'package:flutter_devicelab/tasks/gallery.dart';
88

99
Future<void> main() async {
1010
deviceOperatingSystem = DeviceOperatingSystem.ios;
11+
// ignore: avoid_redundant_argument_values
1112
await task(createGalleryTransitionE2ETest(enableImpeller: true));
1213
}

dev/devicelab/bin/tasks/fullscreen_textfield_perf_impeller_ios__e2e_summary.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,5 +10,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
1010

1111
Future<void> main() async {
1212
deviceOperatingSystem = DeviceOperatingSystem.ios;
13+
// ignore: avoid_redundant_argument_values
1314
await task(createFullscreenTextfieldPerfE2ETest(enableImpeller: true));
1415
}

dev/devicelab/bin/tasks/imagefiltered_transform_animation_perf_impeller_ios__timeline_summary.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
88

99
Future<void> main() async {
1010
deviceOperatingSystem = DeviceOperatingSystem.ios;
11+
// ignore: avoid_redundant_argument_values
1112
await task(createImageFilteredTransformAnimationPerfTest(enableImpeller: true));
1213
}

dev/devicelab/bin/tasks/list_text_layout_impeller_perf__e2e_summary.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
88

99
Future<void> main() async {
1010
deviceOperatingSystem = DeviceOperatingSystem.android;
11+
// ignore: avoid_redundant_argument_values
1112
await task(createListTextLayoutPerfE2ETest(enableImpeller: true));
1213
}

dev/devicelab/bin/tasks/microbenchmarks_impeller_ios.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,5 +9,6 @@ import 'package:flutter_devicelab/tasks/microbenchmarks.dart';
99
/// Runs microbenchmarks on iOS.
1010
Future<void> main() async {
1111
deviceOperatingSystem = DeviceOperatingSystem.ios;
12+
// ignore: avoid_redundant_argument_values
1213
await task(createMicrobenchmarkTask(enableImpeller: true));
1314
}

dev/devicelab/bin/tasks/new_gallery_impeller__transition_perf.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ Future<void> main() async {
1717
final Directory galleryDir = Directory(path.join(galleryParentDir.path, 'gallery'));
1818

1919
try {
20+
// ignore: avoid_redundant_argument_values
2021
await task(NewGalleryPerfTest(galleryDir, enableImpeller: true).run);
2122
} finally {
2223
rmTree(galleryParentDir);

dev/devicelab/bin/tasks/new_gallery_impeller_ios__transition_perf.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ Future<void> main() async {
1717
final Directory galleryDir = Directory(path.join(galleryParentDir.path, 'gallery'));
1818

1919
try {
20+
// ignore: avoid_redundant_argument_values
2021
await task(NewGalleryPerfTest(galleryDir, enableImpeller: true).run);
2122
} finally {
2223
rmTree(galleryParentDir);

dev/devicelab/bin/tasks/platform_views_scroll_perf_impeller_ios__timeline_summary.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
88

99
Future<void> main() async {
1010
deviceOperatingSystem = DeviceOperatingSystem.ios;
11+
// ignore: avoid_redundant_argument_values
1112
await task(createUiKitViewScrollPerfTest(enableImpeller: true));
1213
}

dev/devicelab/bin/tasks/platform_views_scroll_perf_non_intersecting_impeller_ios__timeline_summary.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
88

99
Future<void> main() async {
1010
deviceOperatingSystem = DeviceOperatingSystem.ios;
11+
// ignore: avoid_redundant_argument_values
1112
await task(createUiKitViewScrollPerfNonIntersectingTest(enableImpeller: true));
1213
}

dev/devicelab/bin/tasks/simple_animation_perf_impeller_ios.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
88

99
Future<void> main() async {
1010
deviceOperatingSystem = DeviceOperatingSystem.ios;
11+
// ignore: avoid_redundant_argument_values
1112
await task(createSimpleAnimationPerfTest(enableImpeller: true));
1213
}

dev/devicelab/bin/tasks/tiles_scroll_perf_impeller_ios__timeline_summary.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
88

99
Future<void> main() async {
1010
deviceOperatingSystem = DeviceOperatingSystem.ios;
11+
// ignore: avoid_redundant_argument_values
1112
await task(createTilesScrollPerfTest(enableImpeller: true));
1213
}

dev/devicelab/lib/framework/utils.dart

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@ import 'devices.dart';
1515
import 'host_agent.dart';
1616
import 'task_result.dart';
1717

18+
const bool kEnableImpellerDefault = true;
19+
1820
/// Virtual current working directory, which affect functions, such as [exec].
1921
String cwd = Directory.current.path;
2022

@@ -448,6 +450,9 @@ List<String> _flutterCommandArgs(String command, List<String> options) {
448450
final String? localEngineSrcPath = localEngineSrcPathFromEnv;
449451
return <String>[
450452
command,
453+
if (!command.contains('--enable-impeller')
454+
&& deviceOperatingSystem == DeviceOperatingSystem.ios)
455+
'--enable-impeller',
451456
if (deviceOperatingSystem == DeviceOperatingSystem.ios && supportedDeviceTimeoutCommands.contains(command))
452457
...<String>[
453458
'--device-timeout',

dev/devicelab/lib/tasks/gallery.dart

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ TaskFunction createGalleryTransitionTest({bool semanticsEnabled = false}) {
2828
TaskFunction createGalleryTransitionE2EBuildTest(
2929
List<String> args, {
3030
bool semanticsEnabled = false,
31-
bool enableImpeller = false,
31+
bool enableImpeller = kEnableImpellerDefault,
3232
}) {
3333
return GalleryTransitionBuildTest(
3434
args,
@@ -44,7 +44,7 @@ TaskFunction createGalleryTransitionE2EBuildTest(
4444

4545
TaskFunction createGalleryTransitionE2ETest({
4646
bool semanticsEnabled = false,
47-
bool enableImpeller = false,
47+
bool enableImpeller = kEnableImpellerDefault,
4848
}) {
4949
return GalleryTransitionTest(
5050
testFile: semanticsEnabled
@@ -91,7 +91,7 @@ class GalleryTransitionTest {
9191
this.driverFile,
9292
this.measureCpuGpu = true,
9393
this.measureMemory = true,
94-
this.enableImpeller = false,
94+
this.enableImpeller = kEnableImpellerDefault,
9595
});
9696

9797
final bool semanticsEnabled;
@@ -233,7 +233,7 @@ class GalleryTransitionBuildTest extends BuildTestTask {
233233
this.driverFile,
234234
this.measureCpuGpu = true,
235235
this.measureMemory = true,
236-
this.enableImpeller = false,
236+
this.enableImpeller = kEnableImpellerDefault,
237237
}) : super(workingDirectory: galleryDirectory);
238238

239239
final bool semanticsEnabled;

dev/devicelab/lib/tasks/microbenchmarks.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import '../microbenchmarks.dart';
1515

1616
/// Creates a device lab task that runs benchmarks in
1717
/// `dev/benchmarks/microbenchmarks` reports results to the dashboard.
18-
TaskFunction createMicrobenchmarkTask({bool enableImpeller = false}) {
18+
TaskFunction createMicrobenchmarkTask({bool enableImpeller = kEnableImpellerDefault}) {
1919
return () async {
2020
final Device device = await devices.workingDevice;
2121
await device.unlock();

dev/devicelab/lib/tasks/new_gallery.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ class NewGalleryPerfTest extends PerfTest {
1414
this.galleryDir, {
1515
String timelineFileName = 'transitions',
1616
String dartDefine = '',
17-
bool enableImpeller = false,
17+
bool enableImpeller = kEnableImpellerDefault,
1818
super.timeoutSeconds,
1919
}) : super(
2020
galleryDir.path,

dev/devicelab/lib/tasks/perf_tests.dart

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ String _testOutputDirectory(String testDirectory) {
2626
TaskFunction createComplexLayoutScrollPerfTest({
2727
bool measureCpuGpu = true,
2828
bool badScroll = false,
29-
bool enableImpeller = false,
29+
bool enableImpeller = kEnableImpellerDefault,
3030
}) {
3131
return PerfTest(
3232
'${flutterDirectory.path}/dev/benchmarks/complex_layout',
@@ -39,7 +39,7 @@ TaskFunction createComplexLayoutScrollPerfTest({
3939
).run;
4040
}
4141

42-
TaskFunction createTilesScrollPerfTest({bool enableImpeller = false}) {
42+
TaskFunction createTilesScrollPerfTest({bool enableImpeller = kEnableImpellerDefault}) {
4343
return PerfTest(
4444
'${flutterDirectory.path}/dev/benchmarks/complex_layout',
4545
'test_driver/scroll_perf.dart',
@@ -48,7 +48,7 @@ TaskFunction createTilesScrollPerfTest({bool enableImpeller = false}) {
4848
).run;
4949
}
5050

51-
TaskFunction createUiKitViewScrollPerfTest({bool enableImpeller = false}) {
51+
TaskFunction createUiKitViewScrollPerfTest({bool enableImpeller = kEnableImpellerDefault}) {
5252
return PerfTest(
5353
'${flutterDirectory.path}/dev/benchmarks/platform_views_layout',
5454
'test_driver/uikit_view_scroll_perf.dart',
@@ -59,7 +59,7 @@ TaskFunction createUiKitViewScrollPerfTest({bool enableImpeller = false}) {
5959
).run;
6060
}
6161

62-
TaskFunction createUiKitViewScrollPerfNonIntersectingTest({bool enableImpeller = false}) {
62+
TaskFunction createUiKitViewScrollPerfNonIntersectingTest({bool enableImpeller = kEnableImpellerDefault}) {
6363
return PerfTest(
6464
'${flutterDirectory.path}/dev/benchmarks/platform_views_layout',
6565
'test_driver/uikit_view_scroll_perf_non_intersecting.dart',
@@ -138,7 +138,7 @@ TaskFunction createFlutterGalleryTransitionsPerfSkSLWarmupTest() {
138138

139139
TaskFunction createBackdropFilterPerfTest({
140140
bool measureCpuGpu = true,
141-
bool enableImpeller = false,
141+
bool enableImpeller = kEnableImpellerDefault,
142142
}) {
143143
return PerfTest(
144144
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -182,7 +182,7 @@ TaskFunction createPostBackdropFilterPerfTest({bool measureCpuGpu = true}) {
182182

183183
TaskFunction createSimpleAnimationPerfTest({
184184
bool measureCpuGpu = true,
185-
bool enableImpeller = false,
185+
bool enableImpeller = kEnableImpellerDefault,
186186
}) {
187187
return PerfTest(
188188
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -363,7 +363,7 @@ TaskFunction createFullscreenTextfieldPerfTest() {
363363
}
364364

365365
TaskFunction createFullscreenTextfieldPerfE2ETest({
366-
bool enableImpeller = false,
366+
bool enableImpeller = kEnableImpellerDefault,
367367
}) {
368368
return PerfTest.e2e(
369369
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -389,7 +389,7 @@ TaskFunction createColorFilterAndFadePerfTest() {
389389
).run;
390390
}
391391

392-
TaskFunction createColorFilterAndFadePerfE2ETest({bool enableImpeller = false}) {
392+
TaskFunction createColorFilterAndFadePerfE2ETest({bool enableImpeller = kEnableImpellerDefault}) {
393393
return PerfTest.e2e(
394394
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
395395
'test/color_filter_and_fade_perf_e2e.dart',
@@ -436,7 +436,7 @@ TaskFunction createFadingChildAnimationPerfTest() {
436436
}
437437

438438
TaskFunction createImageFilteredTransformAnimationPerfTest({
439-
bool enableImpeller = false,
439+
bool enableImpeller = kEnableImpellerDefault,
440440
}) {
441441
return PerfTest(
442442
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -455,7 +455,7 @@ TaskFunction createsMultiWidgetConstructPerfE2ETest() {
455455
).run;
456456
}
457457

458-
TaskFunction createListTextLayoutPerfE2ETest({bool enableImpeller = false}) {
458+
TaskFunction createListTextLayoutPerfE2ETest({bool enableImpeller = kEnableImpellerDefault}) {
459459
return PerfTest.e2e(
460460
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
461461
'test/list_text_layout_perf_e2e.dart',
@@ -629,7 +629,7 @@ TaskFunction createGradientStaticPerfE2ETest() {
629629
}
630630

631631
TaskFunction createAnimatedComplexOpacityPerfE2ETest({
632-
bool enableImpeller = false,
632+
bool enableImpeller = kEnableImpellerDefault,
633633
}) {
634634
return PerfTest.e2e(
635635
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -639,7 +639,7 @@ TaskFunction createAnimatedComplexOpacityPerfE2ETest({
639639
}
640640

641641
TaskFunction createAnimatedComplexImageFilteredPerfE2ETest({
642-
bool enableImpeller = false,
642+
bool enableImpeller = kEnableImpellerDefault,
643643
}) {
644644
return PerfTest.e2e(
645645
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -939,7 +939,7 @@ class PerfTest {
939939
String? resultFilename,
940940
this.device,
941941
this.flutterDriveCallback,
942-
this.enableImpeller = false,
942+
this.enableImpeller = kEnableImpellerDefault,
943943
this.timeoutSeconds,
944944
}): _resultFilename = resultFilename;
945945

@@ -955,7 +955,7 @@ class PerfTest {
955955
String resultFilename = 'e2e_perf_summary',
956956
this.device,
957957
this.flutterDriveCallback,
958-
this.enableImpeller = false,
958+
this.enableImpeller = kEnableImpellerDefault,
959959
this.timeoutSeconds,
960960
}) : saveTraceFile = false, timelineFileName = null, _resultFilename = resultFilename;
961961

0 commit comments

Comments
 (0)