fix problem with invented document fragments in number splitting in IE

Author: mikesamuel@gmail.com

js-modules/extractSourceSpans_test.html

@@ -109,7 +109,7 @@ <h1>Extract Source Spans Test</h1>
       }
       actual.innerHTML = '<pre>' + actualHtml + '<\/pre>';
       var goldenText = testResult.innerText || testResult.textContent;
-      var goldenNormalized = '"' + goldenText + '"';
+      var goldenNormalized = '"' + goldenText.replace(/(?:\r\n?|\n)$/, '') + '"';
       var actualNormalized = stringify(actualText);
       var passed = actualNormalized === goldenNormalized;
       if (!passed) {
@@ -125,5 +125,5 @@ <h1>Extract Source Spans Test</h1>
 
 <hr>
 <address></address>
-<!-- hhmts start --> Last modified: Tue Mar 29 16:21:19 PDT 2011 <!-- hhmts end -->
+<!-- hhmts start --> Last modified: Tue Mar 29 16:38:23 PDT 2011 <!-- hhmts end -->
 </body> </html>

js-modules/numberLines.js

@@ -107,8 +107,9 @@ function numberLines(node, opt_startLineNum) {
     var copiedListItem = breakLeftOf(lineEndNode.nextSibling, 0);
 
     // Walk the parent chain until we reach an unattached LI.
-    // Check nodeType since IE sticks useless document fragments around things.
-    for (var parent; (parent = copiedListItem.parentNode) && parent.nodeType === 1;) {
+    for (var parent;
+         // Check nodeType since IE invents document fragments.
+         (parent = copiedListItem.parentNode) && parent.nodeType === 1;) {
       copiedListItem = parent;
     }
     // Put it on the list of lines for later processing.

src/prettify.js

@@ -957,12 +957,16 @@ window['PR']
         return rightSide;
       }
 
-      var split = breakLeftOf(lineEndNode.nextSibling, 0);
+      var copiedListItem = breakLeftOf(lineEndNode.nextSibling, 0);
 
       // Walk the parent chain until we reach an unattached LI.
-      for (var parent; (parent = split.parentNode);) { split = parent; }
+      for (var parent;
+           // Check nodeType since IE invents document fragments.
+           (parent = copiedListItem.parentNode) && parent.nodeType === 1;) {
+        copiedListItem = parent;
+      }
       // Put it on the list of lines for later processing.
-      listItems.push(split);
+      listItems.push(copiedListItem);
     }
 
     // Split lines while there are lines left to split.