Remove HTMLParserOptions

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

 /*
  * Copyright (C) 2010 Google, Inc. All Rights Reserved.
  * Copyright (C) 2011, 2014 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 27 matching lines @@
 #include "core/html/parser/HTMLToken.h"
 #include "core/html/parser/HTMLTokenizer.h"
 
 namespace blink {
 
 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),
 #endif
       m_tree(document)
     , m_insertionMode(HTMLMode)
     , m_originalInsertionMode(HTMLMode)
     , m_parser(parser)
     , m_scriptToProcessStartPosition(uninitializedPositionValue1())
     , m_options(options)
 {
     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)
     // This call makes little sense in fragment mode, but for consistency
     // DocumentParser expects detach() to always be called before it's destroyed.
     m_isAttached = false;
 #endif
     // HTMLConstructionSite might be on the callstack when detach() is called
     // otherwise we'd just call m_tree.clear() here instead.
     m_tree.detach();
@@ skipping 141 matching lines @@
 void HTMLTreeBuilder::finished()
 {
     if (isParsingFragment())
         return;
     ASSERT(m_isAttached);
     // Warning, this may detach the parser. Do not do anything else after this.
     m_tree.finishedParsing();
 }
 
 } // namespace blink