Make popup menus line up to their baseline per the Material spec.

sky/sdk/lib/rendering/object.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 'dart:math' as math;
 import 'dart:sky' as sky;
 import 'dart:sky' show Point, Offset, Size, Rect, Color, Paint, Path;
 
 import '../base/hit_test.dart';
 import '../base/node.dart';
@@ skipping 51 matching lines @@
     assert(child != null);
     setupParentData(child);
     super.adoptChild(child);
     markNeedsLayout();
   }
   void dropChild(RenderObject child) { // only for use by subclasses
     assert(!debugDoingLayout);
     assert(!debugDoingPaint);
     assert(child != null);
     assert(child.parentData != null);
+    child._cleanRelayoutSubtreeRoot();
     child.parentData.detach();
-    child._cleanRelayoutSubtreeRoot();
     super.dropChild(child);
     markNeedsLayout();
   }
 
   static List<RenderObject> _nodesNeedingLayout = new List<RenderObject>();
   static bool _debugDoingLayout = false;
   static bool get debugDoingLayout => _debugDoingLayout;
   bool _debugDoingThisResize = false;
   bool get debugDoingThisResize => _debugDoingThisResize;
   bool _debugDoingThisLayout = false;
@@ skipping 31 matching lines @@
     }
     _needsLayout = true;
     assert(_relayoutSubtreeRoot != null);
     if (_relayoutSubtreeRoot != this) {
       final parent = this.parent; // TODO(ianh): Remove this once the analyzer is cleverer
       assert(parent is RenderObject);
       parent.markNeedsLayout();
       assert(parent == this.parent); // TODO(ianh): Remove this once the analyzer is cleverer
     } else {
       _nodesNeedingLayout.add(this);
-      scheduler.ensureVisualUpdate();
     }
   }
   void _cleanRelayoutSubtreeRoot() {
     if (_relayoutSubtreeRoot != this) {
       _relayoutSubtreeRoot = null;
       _needsLayout = true;
       _cleanRelayoutSubtreeRootChildren();
     }
   }
   void _cleanRelayoutSubtreeRootChildren() { } // workaround for lack of inter-class mixins in Dart
@@ skipping 25 matching lines @@
       assert(_relayoutSubtreeRoot == this);
       _debugCanParentUseSize = false;
       _debugDoingThisLayout = true;
       RenderObject debugPreviousActiveLayout = _debugActiveLayout;
       _debugActiveLayout = this;
       performLayout();
       _debugActiveLayout = debugPreviousActiveLayout;
       _debugDoingThisLayout = false;
       _debugCanParentUseSize = null;
     } catch (e, stack) {
-      print('Exception raised during layout of ${this}: ${e}');
+      print('Exception raised during layout:\n${e}\nContext:\n${this}');
       print(stack);
       return;
     }
     _needsLayout = false;
+    markNeedsPaint();
   }
   void layout(Constraints constraints, { bool parentUsesSize: false }) {
     final parent = this.parent; // TODO(ianh): Remove this once the analyzer is cleverer
     RenderObject relayoutSubtreeRoot;
     if (!parentUsesSize || sizedByParent || constraints.isTight || parent is! RenderObject)
       relayoutSubtreeRoot = this;
     else
       relayoutSubtreeRoot = parent._relayoutSubtreeRoot;
     assert(parent == this.parent); // TODO(ianh): Remove this once the analyzer is cleverer
     if (!needsLayout && constraints == _constraints && relayoutSubtreeRoot == _relayoutSubtreeRoot)
@@ skipping 24 matching lines @@
     // Override this to perform relayout without your parent's
     // involvement.
     //
     // This is called during layout. If sizedByParent is true, then
     // performLayout() should not change your dimensions, only do that
     // in performResize(). If sizedByParent is false, then set both
     // your dimensions and do your children's layout here.
     //
     // When calling layout() on your children, pass in
     // "parentUsesSize: true" if your size or layout is dependent on
-    // your child's size.
+    // your child's size or intrinsic dimensions.
 
   // when the parent has rotated (e.g. when the screen has been turned
   // 90 degrees), immediately prior to layout() being called for the
   // new dimensions, rotate() is called with the old and new angles.
   // The next time paint() is called, the coordinate space will have
   // been rotated N quarter-turns clockwise, where:
   //    N = newAngle-oldAngle
   // ...but the rendering is expected to remain the same, pixel for
   // pixel, on the output device. Then, the layout() method or
   // equivalent will be invoked.
@@ skipping 298 matching lines @@
     int count = 1;
     ChildType child = _firstChild;
     while (child != null) {
       result += '${prefix}child ${count}: ${child.toString(prefix)}';
       count += 1;
       child = child.parentData.nextSibling;
     }
     return result;
   }
 }