Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(34)

Side by Side Diff: LayoutTests/fast/block/lineboxcontain/block-glyphs.html

Issue 1328673003: Remove -webkit-line-box-contain implementation. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Fix up a few more tests. Created 5 years, 3 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
OLDNEW
(Empty)
1 <!doctype html>
2 <head>
3 <title>line-box-contain: block glyphs</title>
4 <style>
5 .test { -webkit-line-box-contain: block glyphs; line-box-contain: block glyphs; font: 64px/1em Ahem; background-color:red; width:6em; overflow:hidden }
6 .goodline { color: green; line-height:12; border-top: 1px solid green; }
7 </style>
8 </head>
9 <body>
10 You should see a green rectangle below. If you see any red, the test has failed .
11
12 <!--
13 Ahem has 0.2em descent and 0.8em ascent, so 64px Ahem has 12.8px descent and 51.2px ascent.
14 Since we expand to enclosing integer points, the integer glyph bounds will b e 65px tall.
15 As a result, we need 64px of text + 1px of extra border to fill the line box .
16 This is not an elegant hack; sorry.
17
18 Warning: this is very sensitive to changes in font code and may need adjustm ent later.
19 -->
20 <div class="test">
21 <span class="goodline">xxxxx</span><span style="background-color:green; font-siz e:128px; padding-left:0.5em;"></span><br>
22 <div style="display:inline-block; width:1em;height:2em;vertical-align:bottom;bac kground-color:green"></div><span class="goodline">xxxxx</span>
23 </div>
24
25 <p style="-webkit-line-box-contain:block glyphs replaced;line-box-contain: block glyphs replaced; border:1px solid black">This line should look normal.</p>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698