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

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

Issue 1746353002: Revert of Fully (?) fix overflow: auto with delayed scroll updates (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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 771acb511c8fb4426626165b72876116e6f5dd5d..ba2878b3f7468b7f66b5582495755e38850c1913 100644
--- a/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
+++ b/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
@@ -612,11 +612,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);
@@ -685,19 +683,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())
@@ -734,8 +727,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