Make the parser an implementation detail of the Document

sky/engine/core/dom/Document.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
@@ skipping 284 matching lines @@
     virtual void detach(const AttachContext& = AttachContext()) override;
     void prepareForDestruction();
 
     // If you have a Document, use renderView() instead which is faster.
     void renderer() const = delete;
 
     RenderView* renderView() const { return m_renderView; }
 
     DocumentLoadTiming* timing() const;
 
-    void startParsing();
+    DocumentParser* startParsing();
     void cancelParsing();
 
     // close() is the DOM API document.close()
     void close(ExceptionState& = ASSERT_NO_EXCEPTION);
     // In some situations (see the code), we ignore document.close().
     // explicitClose() bypass these checks and actually tries to close the
     // input stream.
     void explicitClose();
     // implicitClose() actually does the work of closing the input stream.
     void implicitClose();
@@ skipping 12 matching lines @@
     void setURL(const KURL&);
 
     // To understand how these concepts relate to one another, please see the
     // comments surrounding their declaration.
     const KURL& baseURL() const { return m_baseURL; }
 
     KURL completeURL(const String&) const;
 
     CSSStyleSheet& elementSheet();
 
-    DocumentParser* parser() const { return m_parser.get(); }
-    HTMLDocumentParser* scriptableDocumentParser() const;
+    TextPosition textPosition() const;

[eseidel, 2014/11/12 19:02:14]

What does this mean?

[abarth-chromium, 2014/11/12 21:13:57]

It's the position (row and col) that's currently being parsed.  We should
probably use a better name...

 
     enum ReadyState {
         Loading,
         Interactive,
         Complete
     };
     void setReadyState(ReadyState);
     bool isLoadCompleted();
 
     void setParsing(bool);
@@ skipping 281 matching lines @@
 
     bool needsFullRenderTreeUpdate() const;
 
     bool dirtyElementsForLayerUpdate();
     void updateDistributionIfNeeded();
     void evaluateMediaQueryListIfNeeded();
 
     void updateRenderTree(StyleRecalcChange);
     void updateStyle(StyleRecalcChange);
 
+    HTMLDocumentParser* scriptableDocumentParser() const;
     void detachParser();
 
     virtual bool isDocument() const override final { return true; }
 
     virtual void childrenChanged(const ChildrenChange&) override;
 
     virtual String nodeName() const override final;
     virtual NodeType nodeType() const override final;
     virtual bool childTypeAllowed(NodeType) const override final;
     virtual PassRefPtr<Node> cloneNode(bool deep = true) override final;
@@ skipping 191 matching lines @@
 Node* eventTargetNodeForDocument(Document*);
 
 } // namespace blink
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 void showLiveDocumentInstances();
 #endif
 
 #endif // Document_h