https://bugs.webkit.org/show_bug.cgi?id=126708
Reviewed by Simon Fraser.
Source/WebCore:
I made a typo in r158392 and used was settings the line rect's
height equal to its width. No one noticed because of the subsequent
if statement.
Test: fast/css3-text/css3-text-decoration/text-decoration-skip/text-decoration-skip-tall-underlines.html
* platform/graphics/cg/GraphicsContextCG.cpp:
(WebCore::computeLineBoundsAndAntialiasingModeForText):
LayoutTests:
This test clips to a region that should not be drawn in, then draws a scaled underline. It then compares
the output to an empty element.
* fast/css3-text/css3-text-decoration/text-decoration-skip/text-decoration-skip-tall-underlines-expected.html: Added.
* fast/css3-text/css3-text-decoration/text-decoration-skip/text-decoration-skip-tall-underlines.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@161573
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2014-01-09 Myles C. Maxfield <mmaxfield@apple.com>
+
+ Narrow underlines are too tall
+ https://bugs.webkit.org/show_bug.cgi?id=126708
+
+ Reviewed by Simon Fraser.
+
+ This test clips to a region that should not be drawn in, then draws a scaled underline. It then compares
+ the output to an empty element.
+
+ * fast/css3-text/css3-text-decoration/text-decoration-skip/text-decoration-skip-tall-underlines-expected.html: Added.
+ * fast/css3-text/css3-text-decoration/text-decoration-skip/text-decoration-skip-tall-underlines.html: Added.
+
2014-01-09 Alexey Proskuryakov <ap@apple.com>
REGRESSION (r161429?): Frequent crashes on media/track/media-element-enqueue-event-crash.html
--- /dev/null
+<!DOCTYPE html>
+<html>
+<head>
+</head>
+<body>
+This text draws underlined text with a large scale, then clips the underline to
+a region that should not be drawn. It then makes sure that the image is the same as
+an empty one.
+</body>
+</html>
--- /dev/null
+<!DOCTYPE html>
+<html>
+<head>
+</head>
+<body>
+This text draws underlined text with a large scale, then clips the underline to
+a region that should not be drawn. It then makes sure that the image is the same as
+an empty one.
+<div style="position: relative; overflow: hidden; width: 36px; height: 100px;">
+<div style="left: -160px; top: -320px; position: absolute; -webkit-transform: scale(20); font-family: helvetica; -webkit-transform-origin: left top; display: inline-block; text-decoration: underline; -webkit-text-decoration-skip: ink;">gy</div>
+</div>
+</body>
+</html>
+2014-01-09 Myles C. Maxfield <mmaxfield@apple.com>
+
+ Narrow underlines are too tall
+ https://bugs.webkit.org/show_bug.cgi?id=126708
+
+ Reviewed by Simon Fraser.
+
+ I made a typo in r158392 and used was settings the line rect's
+ height equal to its width. No one noticed because of the subsequent
+ if statement.
+
+ Test: fast/css3-text/css3-text-decoration/text-decoration-skip/text-decoration-skip-tall-underlines.html
+
+ * platform/graphics/cg/GraphicsContextCG.cpp:
+ (WebCore::computeLineBoundsAndAntialiasingModeForText):
+
2014-01-09 Antti Koivisto <antti@apple.com>
DocumentOrderedMap should use iterator
// On screen, use a minimum thickness of 1.0 in user space (later rounded to an integral number in device space).
FloatRect adjustedBounds = initialBounds;
- adjustedBounds.setHeight(std::max(initialBounds.width(), 1.0f));
+ adjustedBounds.setHeight(std::max(initialBounds.height(), 1.0f));
// FIXME: This should be done a better way.
// We try to round all parameters to integer boundaries in device space. If rounding pixels in device space