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

Merge pull request #592 from mpcomplete/animation.asserts

Browse Source 
Fix common asserts in animation API.
This commit is contained in:
mpcomplete 2015-08-13 13:55:28 -04:00
commit dc304b42a6
5 changed files with 12 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
packages/flutter/lib/animation/animated_simulation.dart
View File

@ -51,7 +51,8 @@ class Ticker {
_onTick(timeStamp);
if (isTicking)
// The onTick callback may have scheduled another tick already.
if (isTicking && _animationId == null)
_scheduleTick();
}

2
packages/flutter/lib/animation/timeline.dart
View File

@ -14,7 +14,7 @@ class TweenSimulation extends Simulation {
final double end;
TweenSimulation(Duration duration, this.begin, this.end) :
_durationInSeconds = duration.inMilliseconds / 1000.0 {
_durationInSeconds = duration.inMicroseconds / Duration.MICROSECONDS_PER_SECOND {
assert(_durationInSeconds > 0.0);
assert(begin != null && begin >= 0.0 && begin <= 1.0);
assert(end != null && end >= 0.0 && end <= 1.0);

19
packages/flutter/lib/widgets/drawer.dart
View File

@ -2,7 +2,6 @@
// 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:sky' as sky;
import 'package:sky/animation/animated_value.dart';
@ -69,9 +68,7 @@ class Drawer extends StatefulComponent {
_performance.attachedForce = kDefaultSpringForce;
if (navigator != null) {
scheduleMicrotask(() {
navigator.pushState(this, (_) => _performance.reverse());
});
navigator.pushState(this, (_) => _performance.reverse());
}
}
@ -120,14 +117,12 @@ class Drawer extends StatefulComponent {
}
void _onDismissed() {
scheduleMicrotask(() {
if (navigator != null &&
navigator.currentRoute is RouteState &&
(navigator.currentRoute as RouteState).owner == this) // TODO(ianh): remove cast once analyzer is cleverer
navigator.pop();
if (onDismissed != null)
onDismissed();
});
if (navigator != null &&
navigator.currentRoute is RouteState &&
(navigator.currentRoute as RouteState).owner == this) // TODO(ianh): remove cast once analyzer is cleverer
navigator.pop();
if (onDismissed != null)
onDismissed();
}
bool get _isMostlyClosed => _performance.progress < 0.5;

7
packages/flutter/lib/widgets/popup_menu.dart
View File

@ -2,7 +2,6 @@
// 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:sky' as sky;
import 'package:sky/animation/animated_value.dart';
@ -72,11 +71,7 @@ class PopupMenu extends StatefulComponent {
}
void _open() {
if (navigator != null) {
scheduleMicrotask(() {
navigator.pushState(this, (_) => _close());
});
}
navigator.pushState(this, (_) => _close());
}
void _close() {

3
packages/flutter/lib/widgets/snack_bar.dart
View File

@ -2,7 +2,6 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
import 'dart:async';
import 'package:sky/animation/animated_value.dart';
import 'package:sky/animation/animation_performance.dart';
@ -61,7 +60,7 @@ class SnackBar extends Component {
void _onDismissed() {
if (onDismissed != null)
scheduleMicrotask(() { onDismissed(); });
onDismissed();
}
Widget build() {