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..d94c23ff92a41a8fc0936bb503503f421d281a93 100644
--- a/Source/core/dom/Document.h
+++ b/Source/core/dom/Document.h
@@ -490,11 +490,15 @@ public:
 
     DocumentLoader* loader() const;
 
-    void open(Document* ownerDocument = nullptr, ExceptionState& = ASSERT_NO_EXCEPTION);
+    void open(Document* ownerDocument, ExceptionState&);
     PassRefPtrWillBeRawPtr<DocumentParser> implicitOpen(ParserSynchronizationPolicy);
+    // This is used for internal purpose and does not handle exceptions.

[philipj_slow, 2015/06/24 11:52:19]

s/for internal purpose/internally/

I think also move this up here and in the C++ so that the order matches the flow
order: script-exposed open(), internal open(), implicitOpen().

[Habib Virji, 2015/06/25 13:09:47]

Done.

+    void open();
 
     // close() is the DOM API document.close()
-    void close(ExceptionState& = ASSERT_NO_EXCEPTION);
+    void close(ExceptionState&);
+    // This is used for internal purpose and does not handle exceptions.
+    void close();
     // In some situations (see the code), we ignore document.close().
     // explicitClose() bypass these checks and actually tries to close the
     // input stream.
@@ -518,6 +522,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);
+    void writeln(LocalDOMWindow*, const Vector<String>& text, ExceptionState& = ASSERT_NO_EXCEPTION);
 
     bool wellFormed() const { return m_wellFormed; }