Remove unnecessary includes from HTMLDocumentParser.h

third_party/WebKit/Source/core/html/parser/HTMLDocumentParser.h

 /*
  * Copyright (C) 2010 Google, 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
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  *
  * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY
  * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
  * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL APPLE INC. OR
  * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
  * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
  * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
  * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef HTMLDocumentParser_h
 #define HTMLDocumentParser_h
 
-#include "bindings/core/v8/DocumentWriteEvaluator.h"
 #include "core/dom/ParserContentPolicy.h"
 #include "core/dom/ScriptableDocumentParser.h"
-#include "core/fetch/ResourceClient.h"
-#include "core/frame/UseCounter.h"
 #include "core/html/parser/BackgroundHTMLInputStream.h"
-#include "core/html/parser/CompactHTMLToken.h"

[Charlie Harrison, 2016/08/25 12:55:52]

Looks like we need this one for
    void constructTreeFromCompactHTMLToken(const CompactHTMLToken&);

[pasko, 2016/08/25 13:05:06]

A forward declaration of this class is sufficient, and should be preferred
because all declarations in this file are not using any information about the
internal organization of CompactHTMLToken.

 #include "core/html/parser/HTMLInputStream.h"
 #include "core/html/parser/HTMLParserOptions.h"
 #include "core/html/parser/HTMLParserReentryPermit.h"
 #include "core/html/parser/HTMLPreloadScanner.h"
 #include "core/html/parser/HTMLScriptRunnerHost.h"
 #include "core/html/parser/HTMLSourceTracker.h"
 #include "core/html/parser/HTMLToken.h"
 #include "core/html/parser/HTMLTokenizer.h"
 #include "core/html/parser/HTMLTreeBuilderSimulator.h"
 #include "core/html/parser/ParserSynchronizationPolicy.h"
 #include "core/html/parser/TextResourceDecoder.h"
 #include "core/html/parser/XSSAuditor.h"
 #include "core/html/parser/XSSAuditorDelegate.h"
-#include "platform/text/SegmentedString.h"

[Charlie Harrison, 2016/08/25 12:55:51]

Looks like need this declaration for "void insert(const SegmentedString&) final"

[pasko, 2016/08/25 13:05:06]

Same as above

 #include "wtf/Deque.h"
 #include "wtf/RefPtr.h"
 #include "wtf/WeakPtr.h"
 #include "wtf/text/TextPosition.h"
 #include <memory>
 
 namespace blink {
 
 class BackgroundHTMLParser;
 class CompactHTMLToken;
 class Document;
 class DocumentEncodingData;
 class DocumentFragment;
 class Element;
 class HTMLDocument;
 class HTMLParserScheduler;
+class HTMLPreloadScanner;
 class HTMLResourcePreloader;
 class HTMLScriptRunner;
 class HTMLTreeBuilder;
 class PumpSession;
+class SegmentedString;
 class TokenizedChunkQueue;
+class DocumentWriteEvaluator;
 
 class HTMLDocumentParser :  public ScriptableDocumentParser, private HTMLScriptRunnerHost {
     USING_GARBAGE_COLLECTED_MIXIN(HTMLDocumentParser);
     USING_PRE_FINALIZER(HTMLDocumentParser, dispose);
 public:
     static HTMLDocumentParser* create(HTMLDocument& document, ParserSynchronizationPolicy backgroundParsingPolicy)
     {
         return new HTMLDocumentParser(document, backgroundParsingPolicy);
     }
     ~HTMLDocumentParser() override;
@@ skipping 176 matching lines @@
     bool m_tasksWereSuspended;
     unsigned m_pumpSessionNestingLevel;
     unsigned m_pumpSpeculationsSessionNestingLevel;
     bool m_isParsingAtLineNumber;
     bool m_triedLoadingLinkHeaders;
 };
 
 } // namespace blink
 
 #endif