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

Unified Diff: sky/sdk/lib/framework/layout2.dart

Issue 1157033006: DO NOT COMMIT - fn port to RenderNode, work in progress, does not work (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: fix adam's review comments 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 | « sky/sdk/lib/framework/fn2.dart ('k') | sky/sdk/lib/framework/layouts/block.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/sdk/lib/framework/layout2.dart
diff --git a/sky/sdk/lib/framework/layout2.dart b/sky/sdk/lib/framework/layout2.dart
index e377e9120b6f862c3481cc2764b735331677d8dd..a31c40264ab225f36fc986d13bb2bab92ab37653 100644
--- a/sky/sdk/lib/framework/layout2.dart
+++ b/sky/sdk/lib/framework/layout2.dart
@@ -101,11 +101,12 @@ abstract class RenderNode extends AbstractNode {
return;
}
_needsLayout = true;
- assert(parent is RenderNode);
- if (_relayoutSubtreeRoot != null)
+ if (_relayoutSubtreeRoot != null) {
+ assert(parent is RenderNode);
parent.markNeedsLayout();
- else
+ } else {
_nodesNeedingLayout.add(this);
+ }
}
static void flushLayout() {
_debugDoingLayout = true;
@@ -581,6 +582,8 @@ class RenderView extends RenderNode {
_width = newWidth;
_height = newHeight;
relayout();
+ } else {
+ layoutDone();
}
}
@@ -595,6 +598,7 @@ class RenderView extends RenderNode {
assert(root.width == width);
assert(root.height == height);
}
+ layoutDone();
}
void rotate({ int oldAngle, int newAngle, Duration time }) {
« no previous file with comments | « sky/sdk/lib/framework/fn2.dart ('k') | sky/sdk/lib/framework/layouts/block.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698