Settings menu item in stock2 doesn't work

sky/sdk/lib/app/view.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:sky' as sky;
 
 import '../rendering/box.dart';
 import '../rendering/object.dart';
 import 'scheduler.dart' as scheduler;
 
@@ skipping 24 matching lines @@
       _renderView = renderViewOverride;
     }
     assert(_renderView != null);
 
     assert(_app == this);
   }
 
   static AppView _app; // used to enforce that we're a singleton
 
   RenderView _renderView;
+  RenderView get renderView => _renderView;

[Hixie, 2015/06/17 19:21:25]

I had specifically not made that visible because I don't want people setting the
child directly, rather than through our 'root' setter below. Should we remove
the root setter and have people set it on the renderView directly?

 
   ViewConstraints get _viewConstraints =>
       new ViewConstraints(width: sky.view.width, height: sky.view.height);
 
   Map<int, PointerState> _stateForPointer = new Map<int, PointerState>();
 
   Function onFrame;
 
   RenderBox get root => _renderView.child;
   void set root(RenderBox value) {
@@ skipping 61 matching lines @@
       entry.target.handleEvent(event, entry);
   }
 
   String toString() => 'Render Tree:\n${_renderView}';
 
   void debugDumpRenderTree() {
     toString().split('\n').forEach(print);
   }
   
 }