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

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: Remove the UnknownNonCompositedRegion bucket 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 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 db2ee4b233d8dab7ebc5736b177856ad63136137..7e24325c73e414bef50da88292a5d7338b2e5171 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();
}
@@ -5178,29 +5175,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