I discovered that DocumentFragment is much slower than using Node when it has many childNodes.
Created attachment 42865 [details] proposed patch Removed use of DocumentFragment. Chromium scrolls like butter even while syntax highlighting huge minified files. WebKit Nightly is not much faster. The DOM performance issue occurs for Node.appendChild(Node) too. https://bugs.webkit.org/show_bug.cgi?id=31237 We can bump up the line length limit when this is resolved.
Comment on attachment 42865 [details] proposed patch > - while (node.firstChild) > - node.removeChild(node.firstChild); Why isn't this needed in the loop anymore? (I see removeChildren was added outside the loop.) > - this.lineFragment =null; > + this.newLine =null; Add a space before null. > - action: identOrKeywordAction, > - dontAppendNonToken: true > + action: identOrKeywordAction Why this change? r- for the missing space.
Did you file WebCore bugs about the slow downs?
Created attachment 42871 [details] proposed patch 2 (In reply to comment #2) > (From update of attachment 42865 [details]) > > - while (node.firstChild) > > - node.removeChild(node.firstChild); > > Why isn't this needed in the loop anymore? (I see removeChildren was added > outside the loop.) I removeChildren before the loop and use that directly for this.newLine, eliminating the need for a intermediate node. > > - this.lineFragment =null; > > + this.newLine =null; > > Add a space before null. Fixed. > > - action: identOrKeywordAction, > > - dontAppendNonToken: true > > + action: identOrKeywordAction > > Why this change? I fixed a mistake here. I was going to treat ident as a non-token so dontAppendNonToken was set to true to cut down on the number of TextNodes. However the current code treats both ident and keyword as proper tokens. So I removed the two "this.appendNonToken();" from identOrKeywordAction and set dontAppendNonToken to false so this.appendNonToken will get called inside lex(). > Did you file WebCore bugs about the slow downs? I filed Bug 31237. I'll try to add some more data and a reduced test case.
Comment on attachment 42871 [details] proposed patch 2 Clearing flags on attachment: 42871 Committed r50755: <http://trac.webkit.org/changeset/50755>
All reviewed patches have been landed. Closing bug.