LayoutTests/paint/invalidation/non-stacking-scroller-with-abspos-descendant.html - Issue 1202523006: SP: Invalidate all non-compositing descendants in the slow scroll path.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

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

Side by Side Diff: LayoutTests/paint/invalidation/non-stacking-scroller-with-abspos-descendant.html

Issue 1202523006: SP: Invalidate all non-compositing descendants in the slow scroll path. (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: Created 5 years, 6 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:

OLD	NEW
(Empty)
	1 <!doctype HTML>

	2 <script src="../../resources/run-after-layout-and-paint.js"></script>

	3 <div id="target" style="overflow:scroll; width: 300px; height: 300px;">

	4 <div style="height: 1000px; width: 100px;"></div>

	5 <div style="position: absolute; top: 200px; width: 100px; height: 100px; bac kground: red"></div>

	6 </div>

	7 <script>

	8 // Test that a non-stacking-context scroller does not scroll an absolutely posit ioned child.

	9 function test() {

	10 runAfterLayoutAndPaint(function() {

	11 target.scrollTop += 100;

	12 });
	Xianzhu 2015/06/23 04:56:39 We need a 'true' as the second parameter here, oth We need a 'true' as the second parameter here, otherwise we need explicit waitUntilDone and notifyDone. The crash is because of the pixel capturing after the test automatically finishes. (I have a WIP CL to replace the crash with a warning message) chrishtr 2015/06/23 16:51:37 Done. Show quoted text On 2015/06/23 at 04:56:39, Xianzhu wrote: > We need a 'true' as the second parameter here, otherwise we need explicit waitUntilDone and notifyDone. > > The crash is because of the pixel capturing after the test automatically finishes. (I have a WIP CL to replace the crash with a warning message) Done.
	13 }

	14 window.onload = test;

	15 </script>

OLD	NEW