fixup! fix($compile): correctly merge consecutive text nodes on IE11

Author: gkalpak

src/ng/compile.js

@@ -1850,28 +1850,10 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
 
       for (var i = 0; i < nodeList.length; i++) {
         attrs = new Attributes();
-        node = nodeList[i];
 
         // Workaround for #11781 and #14924
-        if ((msie === 11) && (node.nodeType === NODE_TYPE_TEXT)) {
-          var parent = node.parentNode;
-          var sibling;
-
-          while (true) {
-            sibling = parent ? node.nextSibling : nodeList[i + 1];
-            if (!sibling || sibling.nodeType !== NODE_TYPE_TEXT) {
-              break;
-            }
-
-            node.nodeValue = node.nodeValue + sibling.nodeValue;
-
-            if (sibling.parentNode) {
-              sibling.parentNode.removeChild(sibling);
-            }
-            if (notLiveList && sibling === nodeList[i + 1]) {
-              nodeList.splice(i + 1, 1);
-            }
-          }
+        if (msie === 11) {
+          mergeConsecutiveTextNodes(nodeList, i, notLiveList);
         }
 
         // We must always refer to `nodeList[i]` hereafter,
@@ -1966,6 +1948,32 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
       }
     }
 
+    function mergeConsecutiveTextNodes(nodeList, idx, notLiveList) {
+      var node = nodeList[idx];
+      var parent = node.parentNode;
+      var sibling;
+
+      if (node.nodeType !== NODE_TYPE_TEXT) {
+        return;
+      }
+
+      while (true) {
+        sibling = parent ? node.nextSibling : nodeList[i + 1];
+        if (!sibling || sibling.nodeType !== NODE_TYPE_TEXT) {
+          break;
+        }
+
+        node.nodeValue = node.nodeValue + sibling.nodeValue;
+
+        if (sibling.parentNode) {
+          sibling.parentNode.removeChild(sibling);
+        }
+        if (notLiveList && sibling === nodeList[i + 1]) {
+          nodeList.splice(i + 1, 1);
+        }
+      }
+    }
+
     function createBoundTranscludeFn(scope, transcludeFn, previousBoundTranscludeFn) {
       function boundTranscludeFn(transcludedScope, cloneFn, controllers, futureParentElement, containingScope) {