LayoutTests/fast/block/lineboxcontain/glyphs.html - Issue 1328673003: Remove -webkit-line-box-contain implementation.

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/block/lineboxcontain/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:

Index: LayoutTests/fast/block/lineboxcontain/glyphs.html

diff --git a/LayoutTests/fast/block/lineboxcontain/glyphs.html b/LayoutTests/fast/block/lineboxcontain/glyphs.html

deleted file mode 100644

index 00470be4d7ad011f719d284ce57ebacdba4de158..0000000000000000000000000000000000000000

--- a/LayoutTests/fast/block/lineboxcontain/glyphs.html

+++ /dev/null

@@ -1,30 +0,0 @@

-<!doctype html>

-<head>

-<title>line-box-contain: glyphs</title>

-<style>

-.test { -webkit-line-box-contain: glyphs; line-box-contain: glyphs; font: 64px/1em Ahem; background-color:red; width:6em; overflow:hidden }

-.goodline { color: green; line-height:12; border-top: 1px solid green; }

-.contains-drop-cap::first-letter { font-size:36px; float:left; margin-right:2px; -webkit-line-box-contain:glyphs; line-box-contain:glyphs }

-</style>

-</head>

-<body>

-You should see a green rectangle below. If you see any red, the test has failed.

-<!--

- Ahem has 0.2em descent and 0.8em ascent, so 64px Ahem has 12.8px descent and 51.2px ascent.

- Since we expand to enclosing integer points, the integer glyph bounds will be 65px tall.

- As a result, we need 64px of text + 1px of extra border to fill the line box.

- This is not an elegant hack; sorry.

- Warning: this is very sensitive to changes in font code and may need adjustment later.

--->

-<div class="test">

-<span class="goodline">xxxxx</span><span style="background-color:green; font-size:128px; padding-left:0.5em;"></span><br>

-<div style="display:inline-block; width:1em;height:2em;vertical-align:bottom;background-color:green"></div><span class="goodline">xxxxx</span>

-</div>

-<p style="-webkit-line-box-contain:glyphs;line-box-contain: glyphs; border:1px solid black">this block should hug the lowercase glyphs.</p>

-<p class="contains-drop-cap">The first letter here should look nicer because it uses glyph bounds<br>This should make the top of the T

-line up with the line and not look shifted down.<br> Maybe we should do this by default for all first letters like Firefox does.