Index: Source/core/accessibility/AXScrollView.h |
diff --git a/Source/core/accessibility/AXScrollView.h b/Source/core/accessibility/AXScrollView.h |
index 7df4d9708897c7667673b7a7c6b896576357afa6..caa0963f27e699d2fb6711da1ca64fa4ed14b89f 100644 |
--- a/Source/core/accessibility/AXScrollView.h |
+++ b/Source/core/accessibility/AXScrollView.h |
@@ -75,6 +75,16 @@ private: |
AXScrollbar* addChildScrollbar(Scrollbar*); |
void removeChildScrollbar(AXObject*); |
+ // FIXME: Oilpan: Frame/ScrollView is on the heap and its |
+ // AXScrollView is detached&removed from the AX cache when the |
+ // 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; |