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..c096d919f7755ff4e9a3b306a7023712529f63d0 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);
+    // This is used internally and does not handle exceptions.
+    void open();
+    // open() is the DOM API document.open()

[philipj_slow, 2015/06/27 21:49:24]

Maybe "This is ..." and the same change for close() below, as this otherwise
kind of looks like it's talking about the open() above the comment. I also still
think matching the order to the calling order would make sense, i.e. moving this
and the web-exposed close() up before the internal ones here and in the .cpp

[Habib Virji, 2015/06/29 12:17:42]

Done.

+    void open(Document* ownerDocument, ExceptionState& = ASSERT_NO_EXCEPTION);

[philipj_slow, 2015/06/27 21:49:24]

It doesn't look like this is ever called with just one argument. If so, there
doesn't need to be a default second argument.

[Habib Virji, 2015/06/29 12:17:42]

It was actually but I have removed from here and add it only one place it was
calling.

     PassRefPtrWillBeRawPtr<DocumentParser> implicitOpen(ParserSynchronizationPolicy);
 
+    // This is used internally and does not handle exceptions.
+    void close();
     // close() is the DOM API document.close()
-    void close(ExceptionState& = ASSERT_NO_EXCEPTION);
+    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);
+    void writeln(LocalDOMWindow*, const Vector<String>& text, ExceptionState& = ASSERT_NO_EXCEPTION);
 
     bool wellFormed() const { return m_wellFormed; }