Move attributes and methods from HTMLDocument to Document

Source/core/dom/Document.h

Index: Source/core/dom/Document.h
diff --git a/Source/core/dom/Document.h b/Source/core/dom/Document.h
index f38e4d3abcfe6fe70d2a00464e62b0aa09b4232d..0cbf18000ff66d949097ece43af9fd16f59cc0f3 100644
--- a/Source/core/dom/Document.h
+++ b/Source/core/dom/Document.h
@@ -490,11 +490,16 @@ public:
 
     DocumentLoader* loader() const;
 
-    void open(Document* ownerDocument = nullptr, ExceptionState& = ASSERT_NO_EXCEPTION);
+    // The open and close are used internally for dynamic insertion and does not handle exceptions.

[philipj_slow, 2015/06/29 15:02:38]

Sorry for the churn, but I meant this order of things, here and in
implementation:

// This is the DOM API document.open()
void open(Document* ownerDocument, ExceptionState&);
// This is used internally and does not handle exceptions.
void open();
PassRefPtrWillBeRawPtr<DocumentParser>
implicitOpen(ParserSynchronizationPolicy);

// This is the DOM API document.close()
void close(ExceptionState&);
// This is used internally and does not handle exceptions.
void close();
// In some situations ...

[Habib Virji, 2015/06/29 15:25:21]

Sorry for not getting the order.. Updates now as suggested.

+    void open();
+    void close();
+
+    // This open() is based on the DOM API document.open()
+    void open(Document* ownerDocument, ExceptionState&);
     PassRefPtrWillBeRawPtr<DocumentParser> implicitOpen(ParserSynchronizationPolicy);
 
-    // close() is the DOM API document.close()
-    void close(ExceptionState& = ASSERT_NO_EXCEPTION);
+    // This close() is based on the DOM API document.close()
+    void close(ExceptionState&);
     // In some situations (see the code), we ignore document.close().
     // explicitClose() bypass these checks and actually tries to close the
     // input stream.
@@ -518,6 +523,8 @@ public:
     void write(const SegmentedString& text, Document* ownerDocument = nullptr, ExceptionState& = ASSERT_NO_EXCEPTION);
     void write(const String& text, Document* ownerDocument = nullptr, ExceptionState& = ASSERT_NO_EXCEPTION);
     void writeln(const String& text, Document* ownerDocument = nullptr, ExceptionState& = ASSERT_NO_EXCEPTION);
+    void write(LocalDOMWindow*, const Vector<String>& text, ExceptionState& = ASSERT_NO_EXCEPTION);

[philipj_slow, 2015/06/29 15:02:38]

I don't think these are ever called without the ExceptionState& argument, so can
the default be removed?

[Habib Virji, 2015/06/29 15:25:20]

Done.

+    void writeln(LocalDOMWindow*, const Vector<String>& text, ExceptionState& = ASSERT_NO_EXCEPTION);
 
     bool wellFormed() const { return m_wellFormed; }