Unverified Commit e6bc56f2 authored by Yuqian Li's avatar Yuqian Li Committed by GitHub

Reland "Add Crane test to the new Gallery app (#62162)" (#62544)

parent e5de16a2
// Copyright 2014 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
import 'dart:async';
import 'dart:io';
import 'package:flutter_devicelab/framework/utils.dart';
import 'package:flutter_devicelab/tasks/new_gallery.dart';
import 'package:flutter_devicelab/framework/adb.dart';
import 'package:flutter_devicelab/framework/framework.dart';
import 'package:path/path.dart' as path;
Future<void> main() async {
deviceOperatingSystem = DeviceOperatingSystem.android;
final Directory galleryParentDir =
Directory.systemTemp.createTempSync('new_gallery_test');
final Directory galleryDir =
Directory(path.join(galleryParentDir.path, 'gallery'));
try {
await task(
NewGalleryPerfTest(
galleryDir,
timelineFileName: 'transitions-crane',
dartDefine: 'onlyCrane=true',
).run,
);
} finally {
rmTree(galleryParentDir);
}
}
...@@ -12,7 +12,16 @@ import '../framework/framework.dart'; ...@@ -12,7 +12,16 @@ import '../framework/framework.dart';
import '../framework/utils.dart'; import '../framework/utils.dart';
class NewGalleryPerfTest extends PerfTest { class NewGalleryPerfTest extends PerfTest {
NewGalleryPerfTest(this.galleryDir) : super(galleryDir.path, 'test_driver/transitions_perf.dart', 'transitions'); NewGalleryPerfTest(
this.galleryDir, {
String timelineFileName = 'transitions',
String dartDefine = '',
}) : super(
galleryDir.path,
'test_driver/transitions_perf.dart',
timelineFileName,
dartDefine: dartDefine,
);
@override @override
Future<TaskResult> run() async { Future<TaskResult> run() async {
......
...@@ -357,6 +357,7 @@ class PerfTest { ...@@ -357,6 +357,7 @@ class PerfTest {
this.testDriver, this.testDriver,
this.needsFullTimeline = true, this.needsFullTimeline = true,
this.benchmarkScoreKeys, this.benchmarkScoreKeys,
this.dartDefine = '',
}); });
/// The directory where the app under test is defined. /// The directory where the app under test is defined.
...@@ -395,6 +396,9 @@ class PerfTest { ...@@ -395,6 +396,9 @@ class PerfTest {
/// ``` /// ```
final List<String> benchmarkScoreKeys; final List<String> benchmarkScoreKeys;
/// Additional flags for `--dart-define` to control the test
final String dartDefine;
Future<TaskResult> run() { Future<TaskResult> run() {
return internalRun(); return internalRun();
} }
...@@ -427,6 +431,8 @@ class PerfTest { ...@@ -427,6 +431,8 @@ class PerfTest {
if (writeSkslFileName != null) if (writeSkslFileName != null)
...<String>['--write-sksl-on-exit', writeSkslFileName], ...<String>['--write-sksl-on-exit', writeSkslFileName],
if (cacheSkSL) '--cache-sksl', if (cacheSkSL) '--cache-sksl',
if (dartDefine.isNotEmpty)
...<String>['--dart-define', dartDefine],
'-d', '-d',
deviceId, deviceId,
]); ]);
......
...@@ -812,6 +812,13 @@ tasks: ...@@ -812,6 +812,13 @@ tasks:
stage: devicelab stage: devicelab
required_agent_capabilities: ["mac/ios"] required_agent_capabilities: ["mac/ios"]
new_gallery__crane_perf:
description: >
Measures the performance of the Crane page in the new Flutter Gallery on Android.
stage: devicelab
required_agent_capabilities: ["linux/android"]
flaky: true
fast_scroll_large_images__memory: fast_scroll_large_images__memory:
description: > description: >
Measures memory usage for scrolling through a list of large images. Measures memory usage for scrolling through a list of large images.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment