| Index: Source/core/accessibility/AXScrollView.h
|
| diff --git a/Source/core/accessibility/AXScrollView.h b/Source/core/accessibility/AXScrollView.h
|
| index 4f55902e8c1d0d062afc820ba9a90cb8da609567..77de13a9dd241223d075c548a96aed835356390c 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.
|
| ScrollView* m_scrollView;
|
| RefPtr<AXObject> m_horizontalScrollbar;
|
| RefPtr<AXObject> m_verticalScrollbar;
|
|
|