Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(20)

Side by Side Diff: third_party/WebKit/Source/core/input/ScrollManager.h

Issue 2773593005: Move logic of recording main thread scrolling reasons from cc to blink::ScrollManager (Closed)
Patch Set: Bug fix Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef ScrollManager_h 5 #ifndef ScrollManager_h
6 #define ScrollManager_h 6 #define ScrollManager_h
7 7
8 #include "core/CoreExport.h" 8 #include "core/CoreExport.h"
9 #include "core/page/EventWithHitTestResults.h" 9 #include "core/page/EventWithHitTestResults.h"
10 #include "platform/geometry/LayoutSize.h" 10 #include "platform/geometry/LayoutSize.h"
(...skipping 93 matching lines...) Expand 10 before | Expand all | Expand 10 after
104 104
105 Page* page() const; 105 Page* page() const;
106 106
107 bool isViewportScrollingElement(const Element&) const; 107 bool isViewportScrollingElement(const Element&) const;
108 108
109 bool handleScrollGestureOnResizer(Node*, const WebGestureEvent&); 109 bool handleScrollGestureOnResizer(Node*, const WebGestureEvent&);
110 110
111 void recomputeScrollChain(const Node& startNode, 111 void recomputeScrollChain(const Node& startNode,
112 std::deque<int>& scrollChain); 112 std::deque<int>& scrollChain);
113 113
114 // Record non composited main thread scrolling reasons.
bokan 2017/03/28 15:30:16 Nit: comment unneeded.
115 uint32_t computeNonCompositedMainThreadScrollingReasons();
116 void recordNonCompositedMainThreadScrollingReasons(const WebGestureDevice);
117
114 // NOTE: If adding a new field to this class please ensure that it is 118 // NOTE: If adding a new field to this class please ensure that it is
115 // cleared in |ScrollManager::clear()|. 119 // cleared in |ScrollManager::clear()|.
116 120
117 const Member<LocalFrame> m_frame; 121 const Member<LocalFrame> m_frame;
118 122
119 // Only used with the ScrollCustomization runtime enabled feature. 123 // Only used with the ScrollCustomization runtime enabled feature.
120 std::deque<int> m_currentScrollChain; 124 std::deque<int> m_currentScrollChain;
121 125
122 Member<Node> m_scrollGestureHandlingNode; 126 Member<Node> m_scrollGestureHandlingNode;
123 127
(...skipping 16 matching lines...) Expand all
140 144
141 Member<PaintLayerScrollableArea> m_resizeScrollableArea; 145 Member<PaintLayerScrollableArea> m_resizeScrollableArea;
142 146
143 LayoutSize 147 LayoutSize
144 m_offsetFromResizeCorner; // In the coords of m_resizeScrollableArea. 148 m_offsetFromResizeCorner; // In the coords of m_resizeScrollableArea.
145 }; 149 };
146 150
147 } // namespace blink 151 } // namespace blink
148 152
149 #endif // ScrollManager_h 153 #endif // ScrollManager_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698