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

Unified Diff: third_party/WebKit/Source/core/frame/FrameView.cpp

Issue 2773593005: Move logic of recording main thread scrolling reasons from cc to blink::ScrollManager (Closed)
Patch Set: bug fix Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/frame/FrameView.cpp
diff --git a/third_party/WebKit/Source/core/frame/FrameView.cpp b/third_party/WebKit/Source/core/frame/FrameView.cpp
index 9a608ea2d0aa84af160a63d7440a00ca320aed39..62e3a1e43e217189cb8e9f1bb81ee72e2fea0c8d 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -203,10 +203,7 @@ FrameView::FrameView(LocalFrame& frame)
m_needsScrollbarsUpdate(false),
m_suppressAdjustViewSize(false),
m_allowsLayoutInvalidationAfterLayoutClean(true),
- m_mainThreadScrollingReasons(0),
- m_mainThreadScrollingReasonsCounter(
- MainThreadScrollingReason::kMainThreadScrollingReasonCount,
- 0) {
+ m_mainThreadScrollingReasons(0) {
init();
}
@@ -5162,29 +5159,6 @@ String FrameView::mainThreadScrollingReasonsAsText() const {
return result;
}
-void FrameView::adjustStyleRelatedMainThreadScrollingReasons(
- const uint32_t reason,
- bool increase) {
- int index = MainThreadScrollingReason::getReasonIndex(reason);
- DCHECK_GE(index, 0);
- m_mainThreadScrollingReasonsCounter[index] += increase ? 1 : -1;
- DCHECK_GE(m_mainThreadScrollingReasonsCounter[index], 0);
-}
-
-MainThreadScrollingReasons
-FrameView::getStyleRelatedMainThreadScrollingReasons() const {
- MainThreadScrollingReasons reasons =
- static_cast<MainThreadScrollingReasons>(0);
- for (uint32_t reason = 0;
- reason < MainThreadScrollingReason::kMainThreadScrollingReasonCount;
- ++reason) {
- if (m_mainThreadScrollingReasonsCounter[reason] > 0) {
- reasons |= 1 << reason;
- }
- }
- return reasons;
-}
-
void FrameView::setViewportIntersectionFromParent(
const IntRect& viewportIntersection) {
if (m_remoteViewportIntersection != viewportIntersection) {

Powered by Google App Engine
This is Rietveld 408576698