Oilpan: Replace RefPtrs to Node and its subclasses in core/xml/ with Oilpan transition types.

Source/core/frame/DOMWindow.cpp

Index: Source/core/frame/DOMWindow.cpp
diff --git a/Source/core/frame/DOMWindow.cpp b/Source/core/frame/DOMWindow.cpp
index b9c97b3316aaff6568f0a4ea352bb8318fa83a78..a87b25692d2f0bbca7f353a4d110441a644962d1 100644
--- a/Source/core/frame/DOMWindow.cpp
+++ b/Source/core/frame/DOMWindow.cpp
@@ -363,9 +363,9 @@ void DOMWindow::clearEventQueue()
     m_eventQueue.clear();
 }
 
-PassRefPtr<Document> DOMWindow::createDocument(const String& mimeType, const DocumentInit& init, bool forceXHTML)
+PassRefPtrWillBeRawPtr<Document> DOMWindow::createDocument(const String& mimeType, const DocumentInit& init, bool forceXHTML)
 {
-    RefPtr<Document> document;
+    RefPtrWillBeRawPtr<Document> document;

[haraken, 2014/05/23 05:38:20]

I guess we decided to explicitly initialize this kind of pointer with nullptr,
even if the initialization is covered in the below branches.

[tkent, 2014/05/23 06:21:47]

Done.

     if (forceXHTML) {
         // This is a hack for XSLTProcessor. See XSLTProcessor::createDocumentFromSource().
         document = Document::create(init);
@@ -378,7 +378,7 @@ PassRefPtr<Document> DOMWindow::createDocument(const String& mimeType, const Doc
     return document.release();
 }
 
-PassRefPtr<Document> DOMWindow::installNewDocument(const String& mimeType, const DocumentInit& init, bool forceXHTML)
+PassRefPtrWillBeRawPtr<Document> DOMWindow::installNewDocument(const String& mimeType, const DocumentInit& init, bool forceXHTML)
 {
     ASSERT(init.frame() == m_frame);
 
@@ -388,8 +388,10 @@ PassRefPtr<Document> DOMWindow::installNewDocument(const String& mimeType, const
     m_eventQueue = DOMWindowEventQueue::create(m_document.get());
     m_document->attach();
 
-    if (!m_frame)
-        return m_document;
+    if (!m_frame) {
+        // FIXME: Oilpan: Remove .get() when m_document becomes Member<>.
+        return m_document.get();

[haraken, 2014/05/23 05:38:20]

Document is now on the heap, so can we just make m_document a Member in this CL?

[sof, 2014/05/23 05:43:52]

Not much to it, I had to do this for https://codereview.chromium.org/292503006/
to fix some failing fast/xsl/ tests.

[tkent, 2014/05/23 06:21:47]

I want to avoid it because I don't understand lifetime of DOMWindow and Document
clearly.

[tkent, 2014/05/23 06:21:47]

Wow, conflict!

[sof, 2014/05/23 06:23:51]

I will wait and rebase (there's too many CLs flying in :) )

+    }
 
     m_frame->script().updateDocument();
     m_document->updateViewportDescription();
@@ -410,7 +412,8 @@ PassRefPtr<Document> DOMWindow::installNewDocument(const String& mimeType, const
             m_frame->host()->chrome().client().needTouchEvents(true);
     }
 
-    return m_document;
+    // FIXME: Oilpan: Remove .get() when m_document becomes Member<>.
+    return m_document.get();

[haraken, 2014/05/23 05:38:20]

Ditto.

 }
 
 EventQueue* DOMWindow::eventQueue() const