Unverified Commit 1e8e4147 authored by Michael Goderbauer's avatar Michael Goderbauer Committed by GitHub

Revert "Revert "Migrate some benchmarks to NNBD (#75023)" (#75053)" (#75265)

parent efd48792
...@@ -15,13 +15,11 @@ void main() { ...@@ -15,13 +15,11 @@ void main() {
class PlatformViewApp extends StatefulWidget { class PlatformViewApp extends StatefulWidget {
const PlatformViewApp({ const PlatformViewApp({
Key key Key? key,
}) : super(key: key); }) : super(key: key);
@override @override
PlatformViewAppState createState() => PlatformViewAppState(); PlatformViewAppState createState() => PlatformViewAppState();
static PlatformViewAppState of(BuildContext context) => context.findAncestorStateOfType<PlatformViewAppState>();
} }
class PlatformViewAppState extends State<PlatformViewApp> { class PlatformViewAppState extends State<PlatformViewApp> {
...@@ -42,7 +40,7 @@ class PlatformViewAppState extends State<PlatformViewApp> { ...@@ -42,7 +40,7 @@ class PlatformViewAppState extends State<PlatformViewApp> {
} }
class PlatformViewLayout extends StatelessWidget { class PlatformViewLayout extends StatelessWidget {
const PlatformViewLayout({ Key key }) : super(key: key); const PlatformViewLayout({ Key? key }) : super(key: key);
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
...@@ -72,12 +70,12 @@ class PlatformViewLayout extends StatelessWidget { ...@@ -72,12 +70,12 @@ class PlatformViewLayout extends StatelessWidget {
} }
class DummyPlatformView extends StatelessWidget { class DummyPlatformView extends StatelessWidget {
const DummyPlatformView({Key key}) : super(key: key); const DummyPlatformView({Key? key}) : super(key: key);
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
const String viewType = 'benchmarks/platform_views_layout/DummyPlatformView'; const String viewType = 'benchmarks/platform_views_layout/DummyPlatformView';
Widget nativeView; late Widget nativeView;
if (Platform.isIOS) { if (Platform.isIOS) {
nativeView = const UiKitView( nativeView = const UiKitView(
viewType: viewType, viewType: viewType,
...@@ -98,7 +96,7 @@ class DummyPlatformView extends StatelessWidget { ...@@ -98,7 +96,7 @@ class DummyPlatformView extends StatelessWidget {
} }
class RotationContainer extends StatefulWidget { class RotationContainer extends StatefulWidget {
const RotationContainer({Key key}) : super(key: key); const RotationContainer({Key? key}) : super(key: key);
@override @override
_RotationContainerState createState() => _RotationContainerState(); _RotationContainerState createState() => _RotationContainerState();
...@@ -106,7 +104,7 @@ class RotationContainer extends StatefulWidget { ...@@ -106,7 +104,7 @@ class RotationContainer extends StatefulWidget {
class _RotationContainerState extends State<RotationContainer> class _RotationContainerState extends State<RotationContainer>
with SingleTickerProviderStateMixin { with SingleTickerProviderStateMixin {
AnimationController _rotationController; late AnimationController _rotationController;
@override @override
void initState() { void initState() {
......
...@@ -2,7 +2,7 @@ name: platform_views_layout ...@@ -2,7 +2,7 @@ name: platform_views_layout
description: A benchmark for platform views. description: A benchmark for platform views.
environment: environment:
sdk: ">=2.2.0 <3.0.0" sdk: ">=2.12.0-0 <3.0.0"
dependencies: dependencies:
flutter: flutter:
......
...@@ -7,7 +7,7 @@ import 'package:test/test.dart' hide TypeMatcher, isInstanceOf; ...@@ -7,7 +7,7 @@ import 'package:test/test.dart' hide TypeMatcher, isInstanceOf;
void main() { void main() {
group('scrolling performance test', () { group('scrolling performance test', () {
FlutterDriver driver; late FlutterDriver driver;
setUpAll(() async { setUpAll(() async {
driver = await FlutterDriver.connect(); driver = await FlutterDriver.connect();
...@@ -16,8 +16,7 @@ void main() { ...@@ -16,8 +16,7 @@ void main() {
}); });
tearDownAll(() async { tearDownAll(() async {
if (driver != null) driver.close();
driver.close();
}); });
Future<void> testScrollPerf(String listKey, String summaryName) async { Future<void> testScrollPerf(String listKey, String summaryName) async {
......
...@@ -12,10 +12,9 @@ class AndroidPlatformView extends StatelessWidget { ...@@ -12,10 +12,9 @@ class AndroidPlatformView extends StatelessWidget {
/// Creates a platform view for Android, which is rendered as a /// Creates a platform view for Android, which is rendered as a
/// native view. /// native view.
const AndroidPlatformView({ const AndroidPlatformView({
Key key, Key? key,
@required this.viewType, required this.viewType,
}) : assert(viewType != null), }) : super(key: key);
super(key: key);
/// The unique identifier for the view type to be embedded by this widget. /// The unique identifier for the view type to be embedded by this widget.
/// ///
......
...@@ -19,13 +19,11 @@ void main() { ...@@ -19,13 +19,11 @@ void main() {
class PlatformViewApp extends StatefulWidget { class PlatformViewApp extends StatefulWidget {
const PlatformViewApp({ const PlatformViewApp({
Key key Key? key
}) : super(key: key); }) : super(key: key);
@override @override
PlatformViewAppState createState() => PlatformViewAppState(); PlatformViewAppState createState() => PlatformViewAppState();
static PlatformViewAppState of(BuildContext context) => context.findAncestorStateOfType<PlatformViewAppState>();
} }
class PlatformViewAppState extends State<PlatformViewApp> { class PlatformViewAppState extends State<PlatformViewApp> {
...@@ -46,7 +44,7 @@ class PlatformViewAppState extends State<PlatformViewApp> { ...@@ -46,7 +44,7 @@ class PlatformViewAppState extends State<PlatformViewApp> {
} }
class PlatformViewLayout extends StatelessWidget { class PlatformViewLayout extends StatelessWidget {
const PlatformViewLayout({ Key key }) : super(key: key); const PlatformViewLayout({ Key? key }) : super(key: key);
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
...@@ -76,12 +74,12 @@ class PlatformViewLayout extends StatelessWidget { ...@@ -76,12 +74,12 @@ class PlatformViewLayout extends StatelessWidget {
} }
class DummyPlatformView extends StatelessWidget { class DummyPlatformView extends StatelessWidget {
const DummyPlatformView({Key key}) : super(key: key); const DummyPlatformView({Key? key}) : super(key: key);
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
const String viewType = 'benchmarks/platform_views_layout_hybrid_composition/DummyPlatformView'; const String viewType = 'benchmarks/platform_views_layout_hybrid_composition/DummyPlatformView';
Widget nativeView; late Widget nativeView;
if (Platform.isIOS) { if (Platform.isIOS) {
nativeView = const UiKitView( nativeView = const UiKitView(
viewType: viewType, viewType: viewType,
...@@ -103,7 +101,7 @@ class DummyPlatformView extends StatelessWidget { ...@@ -103,7 +101,7 @@ class DummyPlatformView extends StatelessWidget {
} }
class RotationContainer extends StatefulWidget { class RotationContainer extends StatefulWidget {
const RotationContainer({Key key}) : super(key: key); const RotationContainer({Key? key}) : super(key: key);
@override @override
_RotationContainerState createState() => _RotationContainerState(); _RotationContainerState createState() => _RotationContainerState();
...@@ -111,7 +109,7 @@ class RotationContainer extends StatefulWidget { ...@@ -111,7 +109,7 @@ class RotationContainer extends StatefulWidget {
class _RotationContainerState extends State<RotationContainer> class _RotationContainerState extends State<RotationContainer>
with SingleTickerProviderStateMixin { with SingleTickerProviderStateMixin {
AnimationController _rotationController; late AnimationController _rotationController;
@override @override
void initState() { void initState() {
......
...@@ -2,7 +2,7 @@ name: platform_views_layout_hybrid_composition ...@@ -2,7 +2,7 @@ name: platform_views_layout_hybrid_composition
description: A benchmark for platform views, using hybrid composition on android. description: A benchmark for platform views, using hybrid composition on android.
environment: environment:
sdk: ">=2.2.0 <3.0.0" sdk: ">=2.12.0-0 <3.0.0"
dependencies: dependencies:
flutter: flutter:
......
...@@ -7,7 +7,7 @@ import 'package:test/test.dart' hide TypeMatcher, isInstanceOf; ...@@ -7,7 +7,7 @@ import 'package:test/test.dart' hide TypeMatcher, isInstanceOf;
void main() { void main() {
group('scrolling performance test', () { group('scrolling performance test', () {
FlutterDriver driver; late FlutterDriver driver;
setUpAll(() async { setUpAll(() async {
driver = await FlutterDriver.connect(); driver = await FlutterDriver.connect();
...@@ -16,8 +16,7 @@ void main() { ...@@ -16,8 +16,7 @@ void main() {
}); });
tearDownAll(() async { tearDownAll(() async {
if (driver != null) driver.close();
driver.close();
}); });
Future<void> testScrollPerf(String listKey, String summaryName) async { Future<void> testScrollPerf(String listKey, String summaryName) async {
......
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