LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html - Issue 324733002: Add a layout test for iframe flicker due to overflow scroll promotion

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/compositing/iframes/composited-iframe-transition-flicker.html

Issue 324733002: Add a layout test for iframe flicker due to overflow scroll promotion (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: . Created 6 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:

View unified diff | Download patch | Annotate | Revision Log

OLD	NEW
(Empty)
	1 <!DOCTYPE HTML>

	2 <iframe id="frame" srcdoc="<style>input { width: 100px; }</style><input type='te xt' id='input' value='add stuff here'></input>"></iframe>
	ojan 2014/06/09 17:42:37 Nit: type=text is the default. Nit: type=text is the default. ojan 2014/06/09 17:42:38 Nit: type=text is the default. Nit: type=text is the default. Ian Vollick 2014/06/09 18:35:56 Done. Show quoted text On 2014/06/09 17:42:38, ojan wrote: > Nit: type=text is the default. Done.
	3 <script>

	4 if (window.testRunner) {

	5 testRunner.dumpAsText();
	ojan 2014/06/09 17:42:38 Nit: 4 space indent? Nit: 4 space indent? Ian Vollick 2014/06/09 18:35:56 Done. Show quoted text On 2014/06/09 17:42:38, ojan wrote: > Nit: 4 space indent? Done.
	6 testRunner.waitUntilDone();

	7 }

	8

	9 onload = function() {

	10 var layerTreePreStyleUpdate = "";

	11 var layerTreePostStyleUpdate = "";

	12 if (window.internals)

	13 window.internals.settings.setAcceleratedCompositingForOverflowScrollEnabled( true);

	14

	15 requestAnimationFrame(function() {

	16 document.getElementById("frame").contentDocument.getElementById("input").val ue =

	17 "This is a very long string that will cause overflow";

	18 layerTreePreStyleUpdate = window.internals.layerTreeAsText(document);

	19

	20 // Force a style recalc.

	21 document.body.offsetTop;

	22 layerTreePostStyleUpdate = window.internals.layerTreeAsText(document);

	23

	24 requestAnimationFrame(function() {
	ojan 2014/06/09 17:42:37 I don't think we need the requestAnimationFrame si I don't think we need the requestAnimationFrame since you read both layer trees before. Similarly, this test doesn't need to be async. Ian Vollick 2014/06/09 18:35:56 Done. Show quoted text On 2014/06/09 17:42:37, ojan wrote: > I don't think we need the requestAnimationFrame since you read both layer trees > before. Similarly, this test doesn't need to be async. Done.
	25 var pre = document.createElement('pre');

	26 if (layerTreePreStyleUpdate === layerTreePostStyleUpdate)

	27 pre.innerHTML = "PASS - layer trees are the same.";

	28 else

	29 pre.innerHTML = "FAIL - layer trees are not the same.";

	30 document.body.appendChild(pre);

	31 testRunner.notifyDone();

	32 });

	33 });

	34 };

	35 </script>

OLD	NEW

« no previous file with comments | « no previous file | LayoutTests/compositing/iframes/composited-iframe-transition-flicker-expected.txt » ('j') | no next file with comments »