Move repeated code to removeAllChildren

src/browser/ui/ReactMount.js

@@ -30,6 +30,7 @@ var containsNode = require('containsNode');
 var getReactRootElementInContainer = require('getReactRootElementInContainer');
 var instantiateReactComponent = require('instantiateReactComponent');
 var invariant = require('invariant');
+var removeAllChildren = require('removeAllChildren');
 var setInnerHTML = require('setInnerHTML');
 var shouldUpdateReactComponent = require('shouldUpdateReactComponent');
 var warning = require('warning');
@@ -612,10 +613,7 @@ var ReactMount = {
       container = container.documentElement;
     }
 
-    // http://jsperf.com/emptying-a-node
-    while (container.lastChild) {
-      container.removeChild(container.lastChild);
-    }
+    removeAllChildren(container);
   },
 
   /**

src/browser/ui/dom/removeAllChildren.js

@@ -0,0 +1,25 @@
+/**
+ * Copyright 2013-2015, Facebook, Inc.
+ * All rights reserved.
+ *
+ * This source code is licensed under the BSD-style license found in the
+ * LICENSE file in the root directory of this source tree. An additional grant
+ * of patent rights can be found in the PATENTS file in the same directory.
+ *
+ * @providesModule removeAllChildren
+ */
+
+'use strict';
+
+/**
+ * @param {DOMElement} node
+ */
+function removeAllChildren(node) {
+  // http://jsperf.com/emptying-a-node
+  var firstChild;
+  while ((firstChild = node.firstChild)) {
+    node.removeChild(firstChild);
+  }
+}
+
+module.exports = removeAllChildren;

src/vendor/core/createNodesFromMarkup.js

@@ -17,6 +17,7 @@ var ExecutionEnvironment = require('ExecutionEnvironment');
 var createArrayFrom = require('createArrayFrom');
 var getMarkupWrap = require('getMarkupWrap');
 var invariant = require('invariant');
+var removeAllChildren = require('removeAllChildren');
 
 /**
  * Dummy container used to render all markup.
@@ -77,9 +78,7 @@ function createNodesFromMarkup(markup, handleScript) {
   }
 
   var nodes = createArrayFrom(node.childNodes);
-  while (node.lastChild) {
-    node.removeChild(node.lastChild);
-  }
+  removeAllChildren(node);
   return nodes;
 }