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

Unified Diff: third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp

Issue 1775293002: Revert of Reland "Fully (?) fix overflow: auto with delayed scroll updates (patchset #5 id:80001 of https://c… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
« no previous file with comments | « third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
diff --git a/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp b/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
index bb4e1740f5576c52b35ece5b00568751317bfadf..4e45f19120520db8d6304a7dcc494fb2f75ecf61 100644
--- a/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
+++ b/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
@@ -607,11 +607,9 @@
ScrollableArea::setScrollPosition(newScrollPosition, scrollType, scrollBehavior);
}
-bool PaintLayerScrollableArea::updateAfterLayout(SubtreeLayoutScope* delayedLayoutScope)
+void PaintLayerScrollableArea::updateAfterLayout()
{
ASSERT(box().hasOverflowClip());
-
- bool didMarkForDelayedLayout = false;
if (needsScrollbarReconstruction()) {
m_scrollbarManager.setCanDetachScrollbars(false);
@@ -680,19 +678,14 @@
if ((horizontalScrollBarChanged && box().style()->overflowX() != OOVERLAY) || (verticalScrollBarChanged && box().style()->overflowY() != OOVERLAY)) {
if (!m_inOverflowRelayout) {
m_inOverflowRelayout = true;
- if (delayedLayoutScope) {
- delayedLayoutScope->setNeedsLayout(&box(), LayoutInvalidationReason::ScrollbarChanged);
- didMarkForDelayedLayout = true;
+ SubtreeLayoutScope layoutScope(box());
+ layoutScope.setNeedsLayout(&box(), LayoutInvalidationReason::ScrollbarChanged);
+ if (box().isLayoutBlock()) {
+ LayoutBlock& block = toLayoutBlock(box());
+ block.scrollbarsChanged(horizontalScrollBarChanged, verticalScrollBarChanged);
+ block.layoutBlock(true);
} else {
- SubtreeLayoutScope layoutScope(box());
- layoutScope.setNeedsLayout(&box(), LayoutInvalidationReason::ScrollbarChanged);
- if (box().isLayoutBlock()) {
- LayoutBlock& block = toLayoutBlock(box());
- block.scrollbarsChanged(horizontalScrollBarChanged, verticalScrollBarChanged);
- block.layoutBlock(true);
- } else {
- box().layout();
- }
+ box().layout();
}
LayoutObject* parent = box().parent();
if (parent && parent->isFlexibleBox())
@@ -729,8 +722,6 @@
DisableCompositingQueryAsserts disabler;
positionOverflowControls();
-
- return didMarkForDelayedLayout;
}
ScrollBehavior PaintLayerScrollableArea::scrollBehaviorStyle() const
« no previous file with comments | « third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698