HTMLConstructionSite: avoid n^2 running time for large scripts.

Source/core/html/parser/HTMLTreeBuilder.h

Index: Source/core/html/parser/HTMLTreeBuilder.h
diff --git a/Source/core/html/parser/HTMLTreeBuilder.h b/Source/core/html/parser/HTMLTreeBuilder.h
index d2b541b79ca5fc5e9decc85be9acc69af39c98d6..425678fdd706a118c429729bd2cc0b5a7293c6af 100644
--- a/Source/core/html/parser/HTMLTreeBuilder.h
+++ b/Source/core/html/parser/HTMLTreeBuilder.h
@@ -83,7 +83,7 @@ public:
     void finished();
 
     // Synchronously empty any queues, possibly creating more DOM nodes.
-    void flush() { m_tree.flush(); }
+    void flush() { m_tree.flush(FlushIfAtTextLimit); }

[eseidel, 2014/08/29 08:53:09]

What does this mean now that it doesn't always "flush"?

What does it mean now to flush the treebuilder?

I'm having some difficulty (at least at 2am) paging this code back into my
brain.  The comment above this looks wrong now though.

[eustas, 2014/09/01 09:46:44]

Fixed comment. Pulled "mode" up to actual call-site.

 
     void setShouldSkipLeadingNewline(bool shouldSkip) { m_shouldSkipLeadingNewline = shouldSkip; }