Oilpan: Prepare moving AXObject to heap

Source/modules/accessibility/AXScrollView.h

Index: Source/modules/accessibility/AXScrollView.h
diff --git a/Source/modules/accessibility/AXScrollView.h b/Source/modules/accessibility/AXScrollView.h
index 82842e022e58b1cea9263fe3e18076ab0b87338b..74b559d7129533e90f7122424286fd06598503ad 100644
--- a/Source/modules/accessibility/AXScrollView.h
+++ b/Source/modules/accessibility/AXScrollView.h
@@ -37,11 +37,12 @@ class FrameView;
 
 class AXScrollView final : public AXObject {
 public:
-    static PassRefPtr<AXScrollView> create(FrameView*, AXObjectCacheImpl&);
+    static PassRefPtrWillBeRawPtr<AXScrollView> create(FrameView*, AXObjectCacheImpl&);
     virtual AccessibilityRole roleValue() const override { return ScrollAreaRole; }
     FrameView* scrollView() const { return m_scrollView; }
 
     virtual ~AXScrollView();
+    DECLARE_VIRTUAL_TRACE();
     virtual void detach() override;
 
 protected:
@@ -80,14 +81,9 @@ private:
     // FrameView is disposed. Which clears m_scrollView, hence this
     // bare pointer will not be stale, so the bare pointer use is safe
     // & acceptable.
-    //
-    // However, it would be preferable to have it be normally traced
-    // as part of moving the AX objects to the heap. Temporarily using
-    // a Persistent risks creating a FrameView leak, and brings no
-    // real benefits overall.
-    FrameView* m_scrollView;
-    RefPtr<AXObject> m_horizontalScrollbar;
-    RefPtr<AXObject> m_verticalScrollbar;
+    RawPtrWillBeWeakMember<FrameView> m_scrollView;
+    RefPtrWillBeMember<AXObject> m_horizontalScrollbar;
+    RefPtrWillBeMember<AXObject> m_verticalScrollbar;
     bool m_childrenDirty;
 };