LayoutTests/fast/layout/common-ancestor-relayout-boundary.html - Issue 927773002: Keep track of multiple layout roots

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: LayoutTests/fast/layout/common-ancestor-relayout-boundary.html

Issue 927773002: Keep track of multiple layout roots (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Add bug link Created 5 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:

Index: LayoutTests/fast/layout/common-ancestor-relayout-boundary.html

diff --git a/LayoutTests/fast/layout/common-ancestor-relayout-boundary.html b/LayoutTests/fast/layout/common-ancestor-relayout-boundary.html

index 0c00ebd6677eababc8797c4401098bd495545cbc..3694ff48531b46873e4b18014ee120c370597b19 100644

--- a/LayoutTests/fast/layout/common-ancestor-relayout-boundary.html

+++ b/LayoutTests/fast/layout/common-ancestor-relayout-boundary.html

@@ -31,10 +31,14 @@ if (window.internals)

dirtyLayout();

-// FIXME: In theory we can restrict the layoutRoot in FrameView to the common

-// ancestor container and this should only be 5 instead of 8.

+// The tree appears as following, with the starred nodes dirty:

+// div [relayout-common-ancestor]

+// / \

+// *div *div

+// / /

+// *div *div

if (window.internals)

- shouldBe("window.internals.needsLayoutCount()", "8");

+ shouldBe("window.internals.needsLayoutCount()", "4");

document.documentElement.offsetTop;