Oilpan: Make the Node hierarchy RefCountedGarbageCollected instead of TreeShared.

Source/core/dom/TreeScope.h

Index: Source/core/dom/TreeScope.h
diff --git a/Source/core/dom/TreeScope.h b/Source/core/dom/TreeScope.h
index 9c61b1bebbd832f48714fab105bba36c7d6459de..186615a386061f95e0481f17ba9df429e813ac29 100644
--- a/Source/core/dom/TreeScope.h
+++ b/Source/core/dom/TreeScope.h
@@ -98,7 +98,7 @@ public:
     // Used by the basic DOM mutation methods (e.g., appendChild()).
     void adoptIfNeeded(Node&);
 
-    Node& rootNode() const { return m_rootNode; }
+    Node& rootNode() const { return *m_rootNode; }
 
     IdTargetObserverRegistry& idTargetObserverRegistry() const { return *m_idTargetObserverRegistry.get(); }
 
@@ -157,8 +157,11 @@ protected:
 private:
     virtual void dispose() { }

[haraken, 2014/05/06 15:59:42]

You can add #if !ENABLE(OILPAN) to this method and ShadowRoot::dispose(). In
oilpan, we only need Document::dispose().

[Mads Ager (chromium), 2014/05/07 12:13:16]

I left this one in so I don't have to change the declaration of dispose in
Document. We can ifdef this one out when we can remove dispose on Document as
well.

 
+#if !ENABLE(OILPAN)
     int refCount() const;
-#if SECURITY_ASSERT_ENABLED
+#endif
+
+#if SECURITY_ASSERT_ENABLED && !ENABLE(OILPAN)

[haraken, 2014/05/06 15:59:42]

This should be:

#if !ENABLE(OILPAN)
int refCount() const;
#if SECURITY_ASSERT_ENABLED
...
#else
...
#endif
#endif

[Mads Ager (chromium), 2014/05/07 12:13:16]

Done.

     bool deletionHasBegun();
     void beginDeletion();
 #else
@@ -168,8 +171,8 @@ private:
 
     bool rootNodeHasTreeSharedParent() const;
 
-    Node& m_rootNode;
-    Document* m_document;
+    RawPtrWillBeMember<Node> m_rootNode;
+    RawPtrWillBeMember<Document> m_document;
     RawPtrWillBeMember<TreeScope> m_parentTreeScope;
 
 #if !ENABLE(OILPAN)