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

Unified Diff: sky/framework/fn.dart

Issue 1128803007: [Effen] Move EventListenerNode up so that all the ContentNode subclasses are together in fn.dart. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/framework/fn.dart
diff --git a/sky/framework/fn.dart b/sky/framework/fn.dart
index e9a531af277aa214d1bea3ed66d65486bbdc5008..9378b6bf65eeb5cfa9251f9b6d7fb05bb8a3dd06 100644
--- a/sky/framework/fn.dart
+++ b/sky/framework/fn.dart
@@ -156,56 +156,6 @@ class ParentDataNode extends ContentNode {
ParentDataNode(UINode content, this.parentData): super(content);
}
-/*
- * RenderNodeWrappers correspond to a desired state of a RenderCSS.
- * They are fully immutable, with one exception: A UINode which is a
- * Component which lives within an OneChildListRenderNodeWrapper's
- * children list, may be replaced with the "old" instance if it has
- * become stateful.
- */
-abstract class RenderNodeWrapper extends UINode {
-
- static final Map<RenderCSS, RenderNodeWrapper> _nodeMap =
- new HashMap<RenderCSS, RenderNodeWrapper>();
-
- static RenderNodeWrapper _getMounted(RenderCSS node) => _nodeMap[node];
-
- RenderNodeWrapper({ Object key }) : super(key: key);
-
- RenderNodeWrapper get _emptyNode;
-
- RenderCSS _createNode();
-
- void _sync(UINode old, RenderCSSContainer host, RenderCSS insertBefore) {
- if (old == null) {
- _root = _createNode();
- assert(_root != null);
- host.add(_root, before: insertBefore);
- old = _emptyNode;
- } else {
- _root = old._root;
- assert(_root != null);
- }
-
- _nodeMap[_root] = this;
- _syncNode(old);
- }
-
- void _syncNode(RenderNodeWrapper old);
-
- void _removeChild(UINode node) {
- assert(_root is RenderCSSContainer);
- _root.remove(node._root);
- super._removeChild(node);
- }
-
- void _remove() {
- assert(_root != null);
- _nodeMap.remove(_root);
- super._remove();
- }
-}
-
typedef GestureEventListener(sky.GestureEvent e);
typedef PointerEventListener(sky.PointerEvent e);
typedef EventListener(sky.Event e);
@@ -323,6 +273,56 @@ class EventListenerNode extends ContentNode {
}
}
+/*
+ * RenderNodeWrappers correspond to a desired state of a RenderCSS.
+ * They are fully immutable, with one exception: A UINode which is a
+ * Component which lives within an OneChildListRenderNodeWrapper's
+ * children list, may be replaced with the "old" instance if it has
+ * become stateful.
+ */
+abstract class RenderNodeWrapper extends UINode {
+
+ static final Map<RenderCSS, RenderNodeWrapper> _nodeMap =
+ new HashMap<RenderCSS, RenderNodeWrapper>();
+
+ static RenderNodeWrapper _getMounted(RenderCSS node) => _nodeMap[node];
+
+ RenderNodeWrapper({ Object key }) : super(key: key);
+
+ RenderNodeWrapper get _emptyNode;
+
+ RenderCSS _createNode();
+
+ void _sync(UINode old, RenderCSSContainer host, RenderCSS insertBefore) {
+ if (old == null) {
+ _root = _createNode();
+ assert(_root != null);
+ host.add(_root, before: insertBefore);
+ old = _emptyNode;
+ } else {
+ _root = old._root;
+ assert(_root != null);
+ }
+
+ _nodeMap[_root] = this;
+ _syncNode(old);
+ }
+
+ void _syncNode(RenderNodeWrapper old);
+
+ void _removeChild(UINode node) {
+ assert(_root is RenderCSSContainer);
+ _root.remove(node._root);
+ super._removeChild(node);
+ }
+
+ void _remove() {
+ assert(_root != null);
+ _nodeMap.remove(_root);
+ super._remove();
+ }
+}
+
final List<UINode> _emptyList = new List<UINode>();
abstract class OneChildListRenderNodeWrapper extends RenderNodeWrapper {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698