Right-size the toolbar

sky/sdk/lib/framework/components2/tool_bar.dart

 // Copyright 2015 The Chromium 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 '../fn2.dart';
-import '../theme/view_configuration.dart';
+import '../rendering/flex.dart';
+import '../theme2/view_configuration.dart';
 import '../theme2/shadows.dart';
 
 class ToolBar extends Component {
 
   ToolBar({
     String key,
     this.left,
     this.center,
     this.right,
     this.backgroundColor
@@ skipping 10 matching lines @@
       new FlexExpandingChild(
         new Padding(
           child: center,
           padding: new EdgeDims.only(left: 24.0)
         ))
     ];
 
     if (right != null)
       children.addAll(right);
 
+    var bottomJustifiedChild = new FlexContainer(

[abarth-chromium, 2015/06/10 17:50:40]

Can you add a TODO comment about this being a workaround for the lack of
align-items: flex-end?

[hansmuller, 2015/06/10 17:56:10]

Done.

[Hixie, 2015/06/10 17:57:13]

UINode, not var.

+      direction: FlexDirection.vertical,
+      justifyContent: FlexJustifyContent.flexEnd,
+      children: [new Container(
+          child: new FlexContainer(children: children),
+          height: kToolBarHeight)]);
+
     return new Container(
-      child: new FlexContainer(
-        children: children,
-        direction: FlexDirection.horizontal
-      ),
-      height: 56.0,
-      padding: new EdgeDims(kStatusBarHeight.toDouble(), 8.0, 0.0, 8.0),
+      child: bottomJustifiedChild,
+      padding: new EdgeDims(0.0, 8.0, 0.0, 8.0),

[Hixie, 2015/06/10 17:57:13]

Use the EdgeDims constructor for symmetric sides.

[hansmuller, 2015/06/10 18:06:24]

Done. That's much clearer.

       decoration: new BoxDecoration(
         backgroundColor: backgroundColor,
         boxShadow: Shadow[2]
       )
     );
   }
 
 }