LayoutTests/fast/text-autosizing/inline-width.html - Issue 218863002: [FastTextAutosizer] Fix the inline overlap issue.

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/fast/text-autosizing/inline-width.html

Issue 218863002: [FastTextAutosizer] Fix the inline overlap issue. (Closed) Base URL: svn://svn.chromium.org/blink/trunk

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

	3 <head>

	4

	5 <meta name="viewport" content="width=800">

	6 <style>

	7 html { font-size: x-small; }

	8 .inline { display: inline; font-weight: normal; font-style: normal; background-c olor: green; }
	skobes 2014/03/31 17:19:58 Suggest putting each property on its own line for Suggest putting each property on its own line for readability. pdr. 2014/04/01 06:06:43 Done. Show quoted text On 2014/03/31 17:19:58, skobes wrote: > Suggest putting each property on its own line for readability. Done.
	9 i.inline { background-color: blue; }

	10 b { font-weight: normal; font-style: normal; background-color: purple; }

	11 </style>

	12

	13 <script src="resources/autosizingTest.js"></script>

	14

	15 <script>

	16 onload = function() {
	skobes 2014/03/31 17:19:58 Nit: you can probably inline this at the bottom of Nit: you can probably inline this at the bottom of the <body> instead of using the onload event. pdr. 2014/04/01 06:06:43 Done. Show quoted text On 2014/03/31 17:19:58, skobes wrote: > Nit: you can probably inline this at the bottom of the <body> instead of using > the onload event. Done.
	17 if (window.testRunner)

	18 testRunner.dumpAsText();

	19 var pass = true;

	20 var inlines = document.getElementsByClassName('inline');

	21 var referenceWidth = inlines[0].offsetWidth;

	22 for (var i = 1; i < inlines.length; i++) {

	23 if (inlines[i].offsetWidth != referenceWidth)

	24 pass = false;

	25 }

	26 if (pass)

	27 document.write('PASS: All inlines have the same width.');

	28 else

	29 document.write('FAIL: All inlines do not have the same width.');

	30 }

	31 </script>

	32

	33 </head>

	34 <body>

	35 This test passes if all lines have the same width.<br/>

	36 <div class="inline">

	37 AutosizeAutosizeAutosizeAutosize<br/>

	38 AutosizeAutosizeAutosizeAutosize<br/>

	39 AutosizeAutosizeAutosizeAutosize<br/>

	40 AutosizeAutosizeAutosizeAutosize<br/>

	41 <i class="inline">AutosizeAutosizeAutosizeAutosize</i><br/>

	42 AutosizeAutosizeAutosizeAutosize<br/>

	43 AutosizeAutosizeAutosizeAutosize<br/>

	44 AutosizeAutosizeAutosizeAutosize<br/>

	45 <b><b><b><i class="inline"><b><b><b>AutosizeAutosizeAutosizeAutosize</b></b></b> </i></b></b></b><br/>

	46 AutosizeAutosizeAutosizeAutosize<br/>

	47 AutosizeAutosizeAutosizeAutosize<br/>

	48 </div>

	49 </body>

	50 </html>

OLD	NEW

« no previous file with comments | « no previous file | LayoutTests/fast/text-autosizing/inline-width-expected.txt » ('j') | Source/core/rendering/FastTextAutosizer.cpp » ('J')