Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(381)

Side by Side Diff: sky/sdk/lib/framework/components2/popup_menu_item.dart

Issue 1174023003: Remove one more use of mirrors: Components now have to explicitly sync their fields. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 import '../fn2.dart'; 5 import '../fn2.dart';
6 import 'ink_well.dart'; 6 import 'ink_well.dart';
7 7
8 class PopupMenuItem extends Component { 8 class PopupMenuItem extends Component {
9 PopupMenuItem({ Object key, this.children, this.opacity}) : super(key: key); 9 PopupMenuItem({ Object key, this.children, this.opacity}) : super(key: key);
10 10
11 List<UINode> children; 11 List<UINode> children;
12 double opacity; 12 double opacity;
13 13
14 void syncFields(PopupMenuItem source) {
15 children = source.children;
16 opacity = source.opacity;
17 super.syncFields(source);
18 }
19
14 UINode build() { 20 UINode build() {
15 return new Container( 21 return new Container(
16 constraints: const BoxConstraints(minWidth: 112.0), 22 constraints: const BoxConstraints(minWidth: 112.0),
17 padding: const EdgeDims.all(16.0), 23 padding: const EdgeDims.all(16.0),
18 // TODO(abarth): opacity: opacity, 24 // TODO(abarth): opacity: opacity,
19 child: new BlockContainer(children: children) 25 child: new BlockContainer(children: children)
20 ); 26 );
21 } 27 }
22 } 28 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698