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

Unified Diff: Source/core/rendering/RenderBlock.cpp

Issue 708283002: RLSA needsLayout on customscrollbar thickness change. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: RenderBlock child needsLayout Created 6 years, 1 month 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: Source/core/rendering/RenderBlock.cpp
diff --git a/Source/core/rendering/RenderBlock.cpp b/Source/core/rendering/RenderBlock.cpp
index 01828a11a386acf151a0fd570b615d77a67acf45..f87f190bc92b6a32b9e044368b5a7d18834b4da9 100644
--- a/Source/core/rendering/RenderBlock.cpp
+++ b/Source/core/rendering/RenderBlock.cpp
@@ -1387,9 +1387,15 @@ void RenderBlock::layout()
// Update our first letter info now.
updateFirstLetter();
+ bool relayoutChildren = false;
+
+ // If CustomScrollbar Rect is changed, RenderBlock and child needs reLayout.
+ if (hasOverflowClip())
+ relayoutChildren |= layer()->scrollableArea()->didCustomScrollbarRectChanged();
skobes 2014/11/13 18:14:26 RenderBlockFlow::layoutBlockFlow has logic to forc
MuVen 2014/11/13 20:34:10 I have added a variable in the RenderBox to inform
+
// Table cells call layoutBlock directly, so don't add any logic here. Put code into
// layoutBlock().
- layoutBlock(false);
+ layoutBlock(relayoutChildren);
// It's safe to check for control clip here, since controls can never be table cells.
// If we have a lightweight clip, there can never be any overflow from children.

Powered by Google App Engine
This is Rietveld 408576698