Bug 1328065 - Better document scrollbar-related fields in HitTestingTreeNode. r=kats draft
authorBotond Ballo <botond@mozilla.com>
Fri, 17 Feb 2017 15:42:30 -0500
changeset 486359 ed802693e31fc5064795416a5064c4ad7fa453f3
parent 484877 4c31d9ce488670bb2aab32fcba8e53bba8eaa7cf
child 486360 346cd27d11a570aada70493c3308857b6e960b87
push id45970
push userbballo@mozilla.com
push dateFri, 17 Feb 2017 21:43:56 +0000
reviewerskats
bugs1328065
milestone54.0a1
Bug 1328065 - Better document scrollbar-related fields in HitTestingTreeNode. r=kats MozReview-Commit-ID: 3q3Kgs33qP0
gfx/layers/apz/src/HitTestingTreeNode.h
--- a/gfx/layers/apz/src/HitTestingTreeNode.h
+++ b/gfx/layers/apz/src/HitTestingTreeNode.h
@@ -123,19 +123,25 @@ private:
   RefPtr<HitTestingTreeNode> mPrevSibling;
   RefPtr<HitTestingTreeNode> mParent;
 
   RefPtr<AsyncPanZoomController> mApzc;
   bool mIsPrimaryApzcHolder;
 
   uint64_t mLayersId;
 
+  // This is set for both scroll track and scroll thumb Container layers, and
+  // represents the scroll id of the scroll frame scrolled by the scrollbar.
   FrameMetrics::ViewID mScrollViewId;
+
+  // This is set for scroll thumb Container layers only.
   ScrollDirection mScrollDir;
   int32_t mScrollThumbLength;
+
+  // This is set for scroll track Container layers only.
   bool mIsScrollbarContainer;
 
   FrameMetrics::ViewID mFixedPosTarget;
 
   /* Let {L,M} be the {layer, scrollable metrics} pair that this node
    * corresponds to in the layer tree. mEventRegions contains the event regions
    * from L, in the case where event-regions are enabled. If event-regions are
    * disabled, it will contain the visible region of L, which we use as an