[Impeller] Temporary flag flip for devicelab tests to use Impeller. (#122224)
[Impeller] Temporary flag flip for devicelab tests to use Impeller.
diff --git a/dev/devicelab/bin/tasks/animated_complex_opacity_perf_impeller_ios__e2e_summary.dart b/dev/devicelab/bin/tasks/animated_complex_opacity_perf_impeller_ios__e2e_summary.dart
index 71eacab..b11fc18 100644
--- a/dev/devicelab/bin/tasks/animated_complex_opacity_perf_impeller_ios__e2e_summary.dart
+++ b/dev/devicelab/bin/tasks/animated_complex_opacity_perf_impeller_ios__e2e_summary.dart
@@ -10,5 +10,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createAnimatedComplexOpacityPerfE2ETest(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/backdrop_filter_perf_impeller_ios__timeline_summary.dart b/dev/devicelab/bin/tasks/backdrop_filter_perf_impeller_ios__timeline_summary.dart
index 52ad4ac..fed5a75 100644
--- a/dev/devicelab/bin/tasks/backdrop_filter_perf_impeller_ios__timeline_summary.dart
+++ b/dev/devicelab/bin/tasks/backdrop_filter_perf_impeller_ios__timeline_summary.dart
@@ -8,5 +8,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createBackdropFilterPerfTest(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/color_filter_and_fade_perf_impeller_ios__e2e_summary.dart b/dev/devicelab/bin/tasks/color_filter_and_fade_perf_impeller_ios__e2e_summary.dart
index 682b1e3..e47e6b9 100644
--- a/dev/devicelab/bin/tasks/color_filter_and_fade_perf_impeller_ios__e2e_summary.dart
+++ b/dev/devicelab/bin/tasks/color_filter_and_fade_perf_impeller_ios__e2e_summary.dart
@@ -10,5 +10,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createColorFilterAndFadePerfE2ETest(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/complex_layout_scroll_perf_bad_impeller_ios__timeline_summary.dart b/dev/devicelab/bin/tasks/complex_layout_scroll_perf_bad_impeller_ios__timeline_summary.dart
index 996d04b..18b8032 100644
--- a/dev/devicelab/bin/tasks/complex_layout_scroll_perf_bad_impeller_ios__timeline_summary.dart
+++ b/dev/devicelab/bin/tasks/complex_layout_scroll_perf_bad_impeller_ios__timeline_summary.dart
@@ -8,5 +8,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createComplexLayoutScrollPerfTest(badScroll: true, enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/complex_layout_scroll_perf_impeller_ios__timeline_summary.dart b/dev/devicelab/bin/tasks/complex_layout_scroll_perf_impeller_ios__timeline_summary.dart
index 53578a3..e04f5db 100644
--- a/dev/devicelab/bin/tasks/complex_layout_scroll_perf_impeller_ios__timeline_summary.dart
+++ b/dev/devicelab/bin/tasks/complex_layout_scroll_perf_impeller_ios__timeline_summary.dart
@@ -8,5 +8,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createComplexLayoutScrollPerfTest(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/flutter_gallery__transition_perf_e2e_impeller_ios.dart b/dev/devicelab/bin/tasks/flutter_gallery__transition_perf_e2e_impeller_ios.dart
index 859025e..0f29fea 100644
--- a/dev/devicelab/bin/tasks/flutter_gallery__transition_perf_e2e_impeller_ios.dart
+++ b/dev/devicelab/bin/tasks/flutter_gallery__transition_perf_e2e_impeller_ios.dart
@@ -8,5 +8,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createGalleryTransitionE2ETest(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/fullscreen_textfield_perf_impeller_ios__e2e_summary.dart b/dev/devicelab/bin/tasks/fullscreen_textfield_perf_impeller_ios__e2e_summary.dart
index 5972a38..9d3d213 100644
--- a/dev/devicelab/bin/tasks/fullscreen_textfield_perf_impeller_ios__e2e_summary.dart
+++ b/dev/devicelab/bin/tasks/fullscreen_textfield_perf_impeller_ios__e2e_summary.dart
@@ -10,5 +10,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createFullscreenTextfieldPerfE2ETest(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/imagefiltered_transform_animation_perf_impeller_ios__timeline_summary.dart b/dev/devicelab/bin/tasks/imagefiltered_transform_animation_perf_impeller_ios__timeline_summary.dart
index 51586c0..be27385 100644
--- a/dev/devicelab/bin/tasks/imagefiltered_transform_animation_perf_impeller_ios__timeline_summary.dart
+++ b/dev/devicelab/bin/tasks/imagefiltered_transform_animation_perf_impeller_ios__timeline_summary.dart
@@ -8,5 +8,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createImageFilteredTransformAnimationPerfTest(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/list_text_layout_impeller_perf__e2e_summary.dart b/dev/devicelab/bin/tasks/list_text_layout_impeller_perf__e2e_summary.dart
index 2e4490d..34f8be1 100644
--- a/dev/devicelab/bin/tasks/list_text_layout_impeller_perf__e2e_summary.dart
+++ b/dev/devicelab/bin/tasks/list_text_layout_impeller_perf__e2e_summary.dart
@@ -8,5 +8,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.android;
+ // ignore: avoid_redundant_argument_values
await task(createListTextLayoutPerfE2ETest(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/microbenchmarks_impeller_ios.dart b/dev/devicelab/bin/tasks/microbenchmarks_impeller_ios.dart
index 3daa6d7..7e9ad55 100644
--- a/dev/devicelab/bin/tasks/microbenchmarks_impeller_ios.dart
+++ b/dev/devicelab/bin/tasks/microbenchmarks_impeller_ios.dart
@@ -9,5 +9,6 @@
/// Runs microbenchmarks on iOS.
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createMicrobenchmarkTask(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/new_gallery_impeller__transition_perf.dart b/dev/devicelab/bin/tasks/new_gallery_impeller__transition_perf.dart
index 3ddb155..e3e137b 100644
--- a/dev/devicelab/bin/tasks/new_gallery_impeller__transition_perf.dart
+++ b/dev/devicelab/bin/tasks/new_gallery_impeller__transition_perf.dart
@@ -17,6 +17,7 @@
final Directory galleryDir = Directory(path.join(galleryParentDir.path, 'gallery'));
try {
+ // ignore: avoid_redundant_argument_values
await task(NewGalleryPerfTest(galleryDir, enableImpeller: true).run);
} finally {
rmTree(galleryParentDir);
diff --git a/dev/devicelab/bin/tasks/new_gallery_impeller_ios__transition_perf.dart b/dev/devicelab/bin/tasks/new_gallery_impeller_ios__transition_perf.dart
index a331c4d..394f2e0 100644
--- a/dev/devicelab/bin/tasks/new_gallery_impeller_ios__transition_perf.dart
+++ b/dev/devicelab/bin/tasks/new_gallery_impeller_ios__transition_perf.dart
@@ -17,6 +17,7 @@
final Directory galleryDir = Directory(path.join(galleryParentDir.path, 'gallery'));
try {
+ // ignore: avoid_redundant_argument_values
await task(NewGalleryPerfTest(galleryDir, enableImpeller: true).run);
} finally {
rmTree(galleryParentDir);
diff --git a/dev/devicelab/bin/tasks/platform_views_scroll_perf_impeller_ios__timeline_summary.dart b/dev/devicelab/bin/tasks/platform_views_scroll_perf_impeller_ios__timeline_summary.dart
index 2b8106f..0090863 100644
--- a/dev/devicelab/bin/tasks/platform_views_scroll_perf_impeller_ios__timeline_summary.dart
+++ b/dev/devicelab/bin/tasks/platform_views_scroll_perf_impeller_ios__timeline_summary.dart
@@ -8,5 +8,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createUiKitViewScrollPerfTest(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/platform_views_scroll_perf_non_intersecting_impeller_ios__timeline_summary.dart b/dev/devicelab/bin/tasks/platform_views_scroll_perf_non_intersecting_impeller_ios__timeline_summary.dart
index 3be1c76..346742f 100644
--- a/dev/devicelab/bin/tasks/platform_views_scroll_perf_non_intersecting_impeller_ios__timeline_summary.dart
+++ b/dev/devicelab/bin/tasks/platform_views_scroll_perf_non_intersecting_impeller_ios__timeline_summary.dart
@@ -8,5 +8,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createUiKitViewScrollPerfNonIntersectingTest(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/simple_animation_perf_impeller_ios.dart b/dev/devicelab/bin/tasks/simple_animation_perf_impeller_ios.dart
index 9f6a379..4bee66a 100644
--- a/dev/devicelab/bin/tasks/simple_animation_perf_impeller_ios.dart
+++ b/dev/devicelab/bin/tasks/simple_animation_perf_impeller_ios.dart
@@ -8,5 +8,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createSimpleAnimationPerfTest(enableImpeller: true));
}
diff --git a/dev/devicelab/bin/tasks/tiles_scroll_perf_impeller_ios__timeline_summary.dart b/dev/devicelab/bin/tasks/tiles_scroll_perf_impeller_ios__timeline_summary.dart
index 361df86..9e62ac1 100644
--- a/dev/devicelab/bin/tasks/tiles_scroll_perf_impeller_ios__timeline_summary.dart
+++ b/dev/devicelab/bin/tasks/tiles_scroll_perf_impeller_ios__timeline_summary.dart
@@ -8,5 +8,6 @@
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.ios;
+ // ignore: avoid_redundant_argument_values
await task(createTilesScrollPerfTest(enableImpeller: true));
}
diff --git a/dev/devicelab/lib/framework/utils.dart b/dev/devicelab/lib/framework/utils.dart
index dae7465..938d909 100644
--- a/dev/devicelab/lib/framework/utils.dart
+++ b/dev/devicelab/lib/framework/utils.dart
@@ -15,6 +15,8 @@
import 'host_agent.dart';
import 'task_result.dart';
+const bool kEnableImpellerDefault = true;
+
/// Virtual current working directory, which affect functions, such as [exec].
String cwd = Directory.current.path;
@@ -448,6 +450,9 @@
final String? localEngineSrcPath = localEngineSrcPathFromEnv;
return <String>[
command,
+ if (!command.contains('--enable-impeller')
+ && deviceOperatingSystem == DeviceOperatingSystem.ios)
+ '--enable-impeller',
if (deviceOperatingSystem == DeviceOperatingSystem.ios && supportedDeviceTimeoutCommands.contains(command))
...<String>[
'--device-timeout',
diff --git a/dev/devicelab/lib/tasks/gallery.dart b/dev/devicelab/lib/tasks/gallery.dart
index ea7a07d7..664f76c 100644
--- a/dev/devicelab/lib/tasks/gallery.dart
+++ b/dev/devicelab/lib/tasks/gallery.dart
@@ -28,7 +28,7 @@
TaskFunction createGalleryTransitionE2EBuildTest(
List<String> args, {
bool semanticsEnabled = false,
- bool enableImpeller = false,
+ bool enableImpeller = kEnableImpellerDefault,
}) {
return GalleryTransitionBuildTest(
args,
@@ -44,7 +44,7 @@
TaskFunction createGalleryTransitionE2ETest({
bool semanticsEnabled = false,
- bool enableImpeller = false,
+ bool enableImpeller = kEnableImpellerDefault,
}) {
return GalleryTransitionTest(
testFile: semanticsEnabled
@@ -91,7 +91,7 @@
this.driverFile,
this.measureCpuGpu = true,
this.measureMemory = true,
- this.enableImpeller = false,
+ this.enableImpeller = kEnableImpellerDefault,
});
final bool semanticsEnabled;
@@ -233,7 +233,7 @@
this.driverFile,
this.measureCpuGpu = true,
this.measureMemory = true,
- this.enableImpeller = false,
+ this.enableImpeller = kEnableImpellerDefault,
}) : super(workingDirectory: galleryDirectory);
final bool semanticsEnabled;
diff --git a/dev/devicelab/lib/tasks/microbenchmarks.dart b/dev/devicelab/lib/tasks/microbenchmarks.dart
index 6d35e8a..c53f24d 100644
--- a/dev/devicelab/lib/tasks/microbenchmarks.dart
+++ b/dev/devicelab/lib/tasks/microbenchmarks.dart
@@ -15,7 +15,7 @@
/// Creates a device lab task that runs benchmarks in
/// `dev/benchmarks/microbenchmarks` reports results to the dashboard.
-TaskFunction createMicrobenchmarkTask({bool enableImpeller = false}) {
+TaskFunction createMicrobenchmarkTask({bool enableImpeller = kEnableImpellerDefault}) {
return () async {
final Device device = await devices.workingDevice;
await device.unlock();
diff --git a/dev/devicelab/lib/tasks/new_gallery.dart b/dev/devicelab/lib/tasks/new_gallery.dart
index e7695c8..3532aed 100644
--- a/dev/devicelab/lib/tasks/new_gallery.dart
+++ b/dev/devicelab/lib/tasks/new_gallery.dart
@@ -14,7 +14,7 @@
this.galleryDir, {
String timelineFileName = 'transitions',
String dartDefine = '',
- bool enableImpeller = false,
+ bool enableImpeller = kEnableImpellerDefault,
super.timeoutSeconds,
}) : super(
galleryDir.path,
diff --git a/dev/devicelab/lib/tasks/perf_tests.dart b/dev/devicelab/lib/tasks/perf_tests.dart
index 88fd1bd..32997bd 100644
--- a/dev/devicelab/lib/tasks/perf_tests.dart
+++ b/dev/devicelab/lib/tasks/perf_tests.dart
@@ -26,7 +26,7 @@
TaskFunction createComplexLayoutScrollPerfTest({
bool measureCpuGpu = true,
bool badScroll = false,
- bool enableImpeller = false,
+ bool enableImpeller = kEnableImpellerDefault,
}) {
return PerfTest(
'${flutterDirectory.path}/dev/benchmarks/complex_layout',
@@ -39,7 +39,7 @@
).run;
}
-TaskFunction createTilesScrollPerfTest({bool enableImpeller = false}) {
+TaskFunction createTilesScrollPerfTest({bool enableImpeller = kEnableImpellerDefault}) {
return PerfTest(
'${flutterDirectory.path}/dev/benchmarks/complex_layout',
'test_driver/scroll_perf.dart',
@@ -48,7 +48,7 @@
).run;
}
-TaskFunction createUiKitViewScrollPerfTest({bool enableImpeller = false}) {
+TaskFunction createUiKitViewScrollPerfTest({bool enableImpeller = kEnableImpellerDefault}) {
return PerfTest(
'${flutterDirectory.path}/dev/benchmarks/platform_views_layout',
'test_driver/uikit_view_scroll_perf.dart',
@@ -59,7 +59,7 @@
).run;
}
-TaskFunction createUiKitViewScrollPerfNonIntersectingTest({bool enableImpeller = false}) {
+TaskFunction createUiKitViewScrollPerfNonIntersectingTest({bool enableImpeller = kEnableImpellerDefault}) {
return PerfTest(
'${flutterDirectory.path}/dev/benchmarks/platform_views_layout',
'test_driver/uikit_view_scroll_perf_non_intersecting.dart',
@@ -138,7 +138,7 @@
TaskFunction createBackdropFilterPerfTest({
bool measureCpuGpu = true,
- bool enableImpeller = false,
+ bool enableImpeller = kEnableImpellerDefault,
}) {
return PerfTest(
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -182,7 +182,7 @@
TaskFunction createSimpleAnimationPerfTest({
bool measureCpuGpu = true,
- bool enableImpeller = false,
+ bool enableImpeller = kEnableImpellerDefault,
}) {
return PerfTest(
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -363,7 +363,7 @@
}
TaskFunction createFullscreenTextfieldPerfE2ETest({
- bool enableImpeller = false,
+ bool enableImpeller = kEnableImpellerDefault,
}) {
return PerfTest.e2e(
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -389,7 +389,7 @@
).run;
}
-TaskFunction createColorFilterAndFadePerfE2ETest({bool enableImpeller = false}) {
+TaskFunction createColorFilterAndFadePerfE2ETest({bool enableImpeller = kEnableImpellerDefault}) {
return PerfTest.e2e(
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
'test/color_filter_and_fade_perf_e2e.dart',
@@ -436,7 +436,7 @@
}
TaskFunction createImageFilteredTransformAnimationPerfTest({
- bool enableImpeller = false,
+ bool enableImpeller = kEnableImpellerDefault,
}) {
return PerfTest(
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -455,7 +455,7 @@
).run;
}
-TaskFunction createListTextLayoutPerfE2ETest({bool enableImpeller = false}) {
+TaskFunction createListTextLayoutPerfE2ETest({bool enableImpeller = kEnableImpellerDefault}) {
return PerfTest.e2e(
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
'test/list_text_layout_perf_e2e.dart',
@@ -629,7 +629,7 @@
}
TaskFunction createAnimatedComplexOpacityPerfE2ETest({
- bool enableImpeller = false,
+ bool enableImpeller = kEnableImpellerDefault,
}) {
return PerfTest.e2e(
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -639,7 +639,7 @@
}
TaskFunction createAnimatedComplexImageFilteredPerfE2ETest({
- bool enableImpeller = false,
+ bool enableImpeller = kEnableImpellerDefault,
}) {
return PerfTest.e2e(
'${flutterDirectory.path}/dev/benchmarks/macrobenchmarks',
@@ -939,7 +939,7 @@
String? resultFilename,
this.device,
this.flutterDriveCallback,
- this.enableImpeller = false,
+ this.enableImpeller = kEnableImpellerDefault,
this.timeoutSeconds,
}): _resultFilename = resultFilename;
@@ -955,7 +955,7 @@
String resultFilename = 'e2e_perf_summary',
this.device,
this.flutterDriveCallback,
- this.enableImpeller = false,
+ this.enableImpeller = kEnableImpellerDefault,
this.timeoutSeconds,
}) : saveTraceFile = false, timelineFileName = null, _resultFilename = resultFilename;
