Merge pull request #1259 from abarth/fn3_rebuild

fn3 shouldn't rebuild components that don't change identity

packages/flutter/lib/src/fn3/framework.dart

@@ -204,6 +204,9 @@ abstract class Element<T extends Widget> {
     }
 
     if (child != null) {
+      assert(child._slot == slot);
+      if (child._widget == updated)
+        return child;
       if (_canUpdate(child._widget, updated)) {
         child.update(updated);
         return child;

packages/unit/test/fn3/stateful_component_test.dart

@@ -32,6 +32,21 @@ class TestComponentState extends ComponentState {
 final BoxDecoration kBoxDecorationA = new BoxDecoration();
 final BoxDecoration kBoxDecorationB = new BoxDecoration();
 
+class TestBuildCounter extends Component {
+  static int buildCount = 0;
+
+  Widget build() {
+    ++buildCount;
+    return new DecoratedBox(decoration: kBoxDecorationA);
+  }
+}
+
+void flipStatefulComponent(WidgetTester tester) {
+  ComponentStateElement stateElement =
+      tester.findElement((element) => element is ComponentStateElement);
+  (stateElement.state as TestComponentState).flip();
+}
+
 void main() {
   test('Stateful component smoke test', () {
     WidgetTester tester = new WidgetTester();
@@ -63,10 +78,7 @@ void main() {
 
     checkTree(kBoxDecorationB);
 
-    ComponentStateElement stateElement =
+    flipStatefulComponent(tester);
-        tester.findElement((element) => element is ComponentStateElement);
-    (stateElement.state as TestComponentState).flip();
-
     Element.flushBuild();
 
     checkTree(kBoxDecorationA);
@@ -82,4 +94,20 @@ void main() {
 
   });
 
+  test('Don\'t rebuild subcomponents', () {
+    WidgetTester tester = new WidgetTester();
+    tester.pumpFrame(
+      new TestComponentConfig(
+        left: new TestBuildCounter(),
+        right: new DecoratedBox(decoration: kBoxDecorationB)
+      )
+    );
+
+    expect(TestBuildCounter.buildCount, equals(1));
+
+    flipStatefulComponent(tester);
+    Element.flushBuild();
+
+    expect(TestBuildCounter.buildCount, equals(1));
+  });
 }