Fix issue with syncing render nodes

googlearchive · Sep 27, 2013 · d917377 · d917377
1 parent efaafa3
commit d917377
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/src/ShadowRenderer.js b/src/ShadowRenderer.js
@@ -324,7 +324,7 @@
       }
 
       for (var i = lastIndex; i < newChildren.length; i++) {
-        newChildren[i++].sync(added);
+        newChildren[i].sync(added);
       }
     }
   };

diff --git a/test/js/HTMLContentElement.js b/test/js/HTMLContentElement.js
@@ -88,6 +88,21 @@ suite('HTMLContentElement', function() {
     assertArrayEqual(content.getDistributedNodes(), fooRoot.firstChild.children);
   });
 
+  test('Adding tree with content again', function() {
+    var host = document.createElement('div');
+    host.innerHTML = ' <p>Content</p> ';
+
+    var t = document.createElement('template');
+    t.innerHTML = ' <div> <div> [<content></content>] </div> </div> ';
+
+    var sr = host.createShadowRoot();
+    sr.appendChild(t.content.cloneNode(true));
+
+    host.offsetHeight;
+    assert.equal(unwrap(host).innerHTML,
+        ' <div> <div> [ <p>Content</p> ] </div> </div> ');
+  });
+
   test('adding a new content element to a shadow tree', function() {
     var host = document.createElement('div');
     host.innerHTML = '<a></a><b></b>';