Oilpan: Move all supplements of Page, Document, and WorkerClients to the managed heap.

Source/core/xml/DocumentXPathEvaluator.h

Index: Source/core/xml/DocumentXPathEvaluator.h
diff --git a/Source/core/xml/DocumentXPathEvaluator.h b/Source/core/xml/DocumentXPathEvaluator.h
index 3926518f8e17a0ee6359cd70cadb495abdd43248..18a6a78a9a8fbee1b889e09b0b1c9bf90ece1d9e 100644
--- a/Source/core/xml/DocumentXPathEvaluator.h
+++ b/Source/core/xml/DocumentXPathEvaluator.h
@@ -36,10 +36,9 @@ class ExceptionState;
 class XPathExpression;
 class XPathResult;
 
-class DocumentXPathEvaluator FINAL : public DocumentSupplement {
+class DocumentXPathEvaluator FINAL : public NoBaseWillBeGarbageCollected<DocumentXPathEvaluator>, public DocumentSupplement {
+    WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(DocumentXPathEvaluator);
 public:
-    virtual ~DocumentXPathEvaluator();

[haraken, 2014/04/30 11:18:08]

You can use DECLARE_EMPTY_DESTRUCTOR_WILL_BE_REMOVED.

[zerny-chromium, 2014/04/30 11:46:53]

Again, is it not best to simply remove it now?

-
     static DocumentXPathEvaluator& from(DocumentSupplementable&);
 
     static PassRefPtrWillBeRawPtr<XPathExpression> createExpression(DocumentSupplementable&,
@@ -49,14 +48,14 @@ public:
         const String& expression, Node* contextNode, PassRefPtrWillBeRawPtr<XPathNSResolver>,
         unsigned short type, XPathResult*, ExceptionState&);
 
-    virtual void trace(Visitor*) OVERRIDE { }
+    virtual void trace(Visitor*) OVERRIDE;
 
 private:
     DocumentXPathEvaluator();
 
     static const char* supplementName() { return "DocumentXPathEvaluator"; }
 
-    RefPtrWillBePersistent<XPathEvaluator> m_xpathEvaluator;
+    RefPtrWillBeMember<XPathEvaluator> m_xpathEvaluator;
 };
 
 } // namespace WebCore