Remove HTMLParserOptions

sky/engine/core/html/parser/HTMLTreeBuilder.cpp

Index: sky/engine/core/html/parser/HTMLTreeBuilder.cpp
diff --git a/sky/engine/core/html/parser/HTMLTreeBuilder.cpp b/sky/engine/core/html/parser/HTMLTreeBuilder.cpp
index 1484c82a593b3c3a4ac56beda2f4906b253fdafa..f21c8e6d7bd58e35aa5df16551cb573b2e45a36f 100644
--- a/sky/engine/core/html/parser/HTMLTreeBuilder.cpp
+++ b/sky/engine/core/html/parser/HTMLTreeBuilder.cpp
@@ -45,7 +45,7 @@ static TextPosition uninitializedPositionValue1()
     return TextPosition(OrdinalNumber::fromOneBasedInt(-1), OrdinalNumber::first());
 }
 
-HTMLTreeBuilder::HTMLTreeBuilder(HTMLDocumentParser* parser, HTMLDocument* document, bool, const HTMLParserOptions& options)
+HTMLTreeBuilder::HTMLTreeBuilder(HTMLDocumentParser* parser, HTMLDocument* document)
     :
 #if ENABLE(ASSERT)
       m_isAttached(true),
@@ -60,43 +60,10 @@ HTMLTreeBuilder::HTMLTreeBuilder(HTMLDocumentParser* parser, HTMLDocument* docum
     m_tree.openElements()->pushRootNode(document);
 }
 
-// FIXME: Member variables should be grouped into self-initializing structs to
-// minimize code duplication between these constructors.
-HTMLTreeBuilder::HTMLTreeBuilder(HTMLDocumentParser* parser, DocumentFragment* fragment, Element* contextElement, const HTMLParserOptions& options)

[eseidel, 2014/10/25 05:08:38]

Do we no longer parse fragments?  If not, we can remove a ton of other fragment
code.

-    :
-#if ENABLE(ASSERT)
-     m_isAttached(true),
-#endif
-     m_fragmentContext(fragment, contextElement)
-    , m_tree(fragment)
-    , m_insertionMode(HTMLMode)
-    , m_originalInsertionMode(HTMLMode)
-    , m_parser(parser)
-    , m_scriptToProcessStartPosition(uninitializedPositionValue1())
-    , m_options(options)
-{
-    ASSERT(isMainThread());
-    ASSERT(contextElement);
-
-    // Steps 4.2-4.6 of the HTML5 Fragment Case parsing algorithm:
-    // http://www.whatwg.org/specs/web-apps/current-work/multipage/the-end.html#fragment-case
-    // For efficiency, we skip step 4.2 ("Let root be a new html element with no attributes")
-    // and instead use the DocumentFragment as a root node.
-    m_tree.openElements()->pushRootNode(fragment);
-}
-
 HTMLTreeBuilder::~HTMLTreeBuilder()
 {
 }
 
-void HTMLTreeBuilder::trace(Visitor* visitor)
-{
-    visitor->trace(m_fragmentContext);
-    visitor->trace(m_tree);
-    visitor->trace(m_parser);
-    visitor->trace(m_scriptToProcess);
-}
-
 void HTMLTreeBuilder::detach()
 {
 #if ENABLE(ASSERT)