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

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: Update recording logic for non composited reasons 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 5f4f5758e0fa9f5d417d334b3964da60f1a4635b..06bd3eb0118231a67f507a7a4310e041a1f7e9e4 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -204,10 +204,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();
}
@@ -5176,29 +5173,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