LayoutTests/fast/text-autosizing/oscillation-javascript-fontsize-change.html - Issue 356133003: Update the two remaining tests for the fast text autosizer

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/text-autosizing/oscillation-javascript-fontsize-change.html

Issue 356133003: Update the two remaining tests for the fast text autosizer (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 side-by-side diff with in-line comments

Download patch

« no previous file with comments | « LayoutTests/fast/text-autosizing/cluster-list-item-expected.html ('k') | LayoutTests/fast/text-autosizing/oscillation-javascript-fontsize-change-expected.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: LayoutTests/fast/text-autosizing/oscillation-javascript-fontsize-change.html

diff --git a/LayoutTests/fast/text-autosizing/oscillation-javascript-fontsize-change.html b/LayoutTests/fast/text-autosizing/oscillation-javascript-fontsize-change.html

index 741bd6ff678934766b683820e26d96d19fecfb4d..8e39e47b9e3db7eff8e5e3b1d6f36930218785f0 100644

--- a/LayoutTests/fast/text-autosizing/oscillation-javascript-fontsize-change.html

+++ b/LayoutTests/fast/text-autosizing/oscillation-javascript-fontsize-change.html

@@ -1,7 +1,6 @@

<!DOCTYPE html>

<head>

<style>

body {

@@ -11,36 +10,27 @@

}

.largersize{font-size: 1.1em}

</style>

</head>

<body>

-<div style="top: 50px; position: absolute; overflow: auto;">

- . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . .<br/>

- This paragraph should be autosized to 19.8px<br/>

+<div>

+ This paragraph should be autosized to 40px<br/>

because it contains line breaks.<br/>

This test is intended to check<br/>

that there are no oscillations due to javascript<br/>

briefly increasing the font size of a<br/>

small paragraph below.

</div>

-<div id="sizechanging">

+<div id="sizechanging" style="position: absolute; bottom: 0;">

This text changes size using javascript below.

</div>

-element = document.getElementById("sizechanging");

-if (element.offsetHeight) {

- // force layout (computation of offsetHeight triggers reflow)

-element.className = 'largersize';

-if (element.offsetHeight) {}

-element.className = '';

-if (element.offsetHeight) {}

+ var element = document.getElementById("sizechanging");

+ var forceLayout = element.offsetHeight;

+ element.className = 'largersize';

+ forceLayout = element.offsetHeight;

+ element.className = '';

+ forceLayout = element.offsetHeight;

</script>

-<noscript>fail (no support for javascript)</noscript>

</body>

</html>