From 0866485f951301fbe7ec236f693f4cf5dd368afb Mon Sep 17 00:00:00 2001
From: Misko Hevery <misko@hevery.com>
Date: Thu, 11 Dec 2014 16:07:49 -0800
Subject: [PATCH] perf: Traverse dom using firstChild instead of childNodes

5% improvement in speed by not using childNodes DOM API.
---
 modules/core/src/compiler/view.js | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/modules/core/src/compiler/view.js b/modules/core/src/compiler/view.js
index 8dcaccb7d8..61c7fcd8f7 100644
--- a/modules/core/src/compiler/view.js
+++ b/modules/core/src/compiler/view.js
@@ -275,11 +275,12 @@ export class ProtoView {
     }
     var viewNodes;
     if (this.isTemplateElement) {
-      var childNodes = rootElementClone.content.childNodes;
-      // Note: An explicit loop is the fastes way to convert a DOM array into a JS array!
-      viewNodes = ListWrapper.createFixedSize(childNodes.length);
-      for (var i=0; i<childNodes.length; i++) {
-        viewNodes[i] = childNodes[i];
+      var childNode = DOM.firstChild(rootElementClone.content);
+      viewNodes = []; // TODO(perf): Should be fixed size, since we could pre-compute in in ProtoView
+      // Note: An explicit loop is the fastest way to convert a DOM array into a JS array!
+      while(childNode != null) {
+        ListWrapper.push(viewNodes, childNode);
+        childNode = DOM.nextSibling(childNode);
       }
     } else {
       viewNodes = [rootElementClone];
@@ -338,9 +339,12 @@ export class ProtoView {
       // textNodes
       var textNodeIndices = binder.textNodeIndices;
       if (isPresent(textNodeIndices)) {
-        var childNodes = DOM.templateAwareRoot(element).childNodes;
-        for (var j = 0; j < textNodeIndices.length; j++) {
-          ListWrapper.push(textNodes, childNodes[textNodeIndices[j]]);
+        var childNode = DOM.firstChild(DOM.templateAwareRoot(element));
+        for (var j = 0, k = 0; j < textNodeIndices.length; j++) {
+          for(var index = textNodeIndices[j]; k < index; k++) {
+            childNode = DOM.nextSibling(childNode);
+          }
+          ListWrapper.push(textNodes, childNode);
         }
       }