firka/flutter
firka/flutter
4
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Move -referencing asserts to constructor body. (#12478)

Browse Source
This commit is contained in:
Alexander Aprelev 2017-10-10 09:05:20 -07:00 committed by GitHub
parent 7f5b973cb1
commit c38b843460
3 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
packages/flutter/lib/src/cupertino/route.dart
View File

@ -88,8 +88,9 @@ class CupertinoPageRoute<T> extends PageRoute<T> {
assert(settings != null), assert(settings != null),
assert(maintainState != null), assert(maintainState != null),
assert(fullscreenDialog != null), assert(fullscreenDialog != null),
assert(opaque), // PageRoute makes it return true. super(settings: settings, fullscreenDialog: fullscreenDialog) {
super(settings: settings, fullscreenDialog: fullscreenDialog); assert(opaque); // PageRoute makes it return true.
}
/// Builds the primary contents of the route. /// Builds the primary contents of the route.
final WidgetBuilder builder; final WidgetBuilder builder;

5
packages/flutter/lib/src/material/page.dart
View File

@ -70,8 +70,9 @@ class MaterialPageRoute<T> extends PageRoute<T> {
this.maintainState: true, this.maintainState: true,
bool fullscreenDialog: false, bool fullscreenDialog: false,
}) : assert(builder != null), }) : assert(builder != null),
assert(opaque), super(settings: settings, fullscreenDialog: fullscreenDialog) {
super(settings: settings, fullscreenDialog: fullscreenDialog); assert(opaque);
}
/// Builds the primary contents of the route. /// Builds the primary contents of the route.
final WidgetBuilder builder; final WidgetBuilder builder;

5
packages/flutter/lib/src/widgets/pages.dart
View File

@ -81,10 +81,11 @@ class PageRouteBuilder<T> extends PageRoute<T> {
this.maintainState: true, this.maintainState: true,
}) : assert(pageBuilder != null), }) : assert(pageBuilder != null),
assert(transitionsBuilder != null), assert(transitionsBuilder != null),
assert(opaque != null),
assert(barrierDismissible != null), assert(barrierDismissible != null),
assert(maintainState != null), assert(maintainState != null),
super(settings: settings); super(settings: settings) {
assert(opaque != null);
}
/// Used build the route's primary contents. /// Used build the route's primary contents.
/// ///