SkyView should handle screen rotation

sky/sdk/lib/framework/app.dart

Index: sky/sdk/lib/framework/app.dart
diff --git a/sky/sdk/lib/framework/app.dart b/sky/sdk/lib/framework/app.dart
index 2c84da438bff732c9b54c89fb83cf4d54d32cfb7..7826cf8ca776770da97e9639ce522af2f34ba0f3 100644
--- a/sky/sdk/lib/framework/app.dart
+++ b/sky/sdk/lib/framework/app.dart
@@ -18,19 +18,20 @@ class AppView {
 
   AppView(RenderBox root) {
     sky.view.setEventCallback(_handleEvent);
+    sky.view.setMetricsChangedCallback(_handleMetricsChanged);
     scheduler.init();
     scheduler.addPersistentFrameCallback(_beginFrame);
 
     _renderView = new RenderView(child: root);
     _renderView.attach();
-    _renderView.layout(new ViewConstraints(width: sky.view.width,
-                                           height: sky.view.height));
-
-    scheduler.ensureVisualUpdate();
+    _renderView.layout(_viewConstraints);
   }
 
   RenderView _renderView;
 
+  ViewConstraints get _viewConstraints =>
+      new ViewConstraints(width: sky.view.width, height: sky.view.height);
+
   Map<int, PointerState> _stateForPointer = new Map<int, PointerState>();
 
   Function onFrame;
@@ -56,6 +57,11 @@ class AppView {
     }
   }
 
+  void _handleMetricsChanged() {
+    if (_renderView != null)
+      _renderView.layout(_viewConstraints);

[Hixie, 2015/06/09 20:31:31]

I'm surprised this doesn't assert. We're not in debugDoingLayout here, yet this
is going to adjust the position and size of the whole tree.

+  }
+
   PointerState _createStateForPointer(sky.PointerEvent event, Point position) {
     HitTestResult result = new HitTestResult();
     _renderView.hitTest(result, position: position);