| 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;
|
|
|