Simplify the Drawer animation

Simplify the Drawer animation

Instead of using AnimationBuilder, this CL has the Drawer just
use the animations system directly.

R=mpcomplete@chromium.org, mpcomplete@google.com

Committed: https://chromium.googlesource.com/external/mojo/+/c6d8cf07bc85a23ef6c1a33b1bf090c70a02ba10

[Matt Perry, 2015-07-15 00:20:18]

mpcomplete@chromium.org changed reviewers:
+ mpcomplete@chromium.org

[Matt Perry, 2015-07-15 00:20:18]

I'd be curious to see what Material looks like under this model.

https://codereview.chromium.org/1234073005/diff/1/sky/sdk/lib/widgets/animate...
File sky/sdk/lib/widgets/animated.dart (right):

https://codereview.chromium.org/1234073005/diff/1/sky/sdk/lib/widgets/animate...
sky/sdk/lib/widgets/animated.dart:11: class PositionAnimation extends
AnimatedComponent {
nit: I like AnimatedFoo convention for animated components

https://codereview.chromium.org/1234073005/diff/1/sky/sdk/lib/widgets/animate...
sky/sdk/lib/widgets/animated.dart:64: decoration: new
BoxDecoration(backgroundColor: color.value),
One of the problems I ran into with animating Material was animating 2 pieces of
a Container. A shadowed Container within a bordered Container is different than
a single Container with shadow+border. Are you planning on adding a
ContainerAnimation?

[Matt Perry, 2015-07-15 00:22:04]

LGTM

[abarth-chromium, 2015-07-15 00:23:53]

https://codereview.chromium.org/1234073005/diff/1/sky/sdk/lib/widgets/animate...
File sky/sdk/lib/widgets/animated.dart (right):

https://codereview.chromium.org/1234073005/diff/1/sky/sdk/lib/widgets/animate...
sky/sdk/lib/widgets/animated.dart:64: decoration: new
BoxDecoration(backgroundColor: color.value),
On 2015/07/15 at 00:20:18, Matt Perry wrote:
> One of the problems I ran into with animating Material was animating 2 pieces
of a Container. A shadowed Container within a bordered Container is different
than a single Container with shadow+border. Are you planning on adding a
ContainerAnimation?

Yeah, IMHO ColorAnimation should grow to be ContainerAnimation.

[abarth-chromium, 2015-07-15 03:36:23]

Committed patchset #2 (id:20001) manually as
c6d8cf07bc85a23ef6c1a33b1bf090c70a02ba10 (presubmit successful).