Changeset 178647 in webkit

Timestamp:

Jan 19, 2015 8:32:26 AM (9 years ago)

Author:

mmaxfield@apple.com

Message:

[SVG -> OTF Converter] Glyphs get clipped weirdly
​https://bugs.webkit.org/show_bug.cgi?id=137095

Reviewed by Antti Koivisto.

The Adobe CFF spec doesn't actually tell you how to serialize a
"FontBBox." After trial and error, it seems to be (x, y, width,
height).

Test: svg/text/kerning.svg

svg/W3C-SVG-1.1/fonts-kern-01-t.svg

• svg/SVGToOTFFontConversion.cpp:

(WebCore::SVGToOTFFontConverter::appendHEADTable):
(WebCore::SVGToOTFFontConverter::appendCFFTable):
(WebCore::SVGToOTFFontConverter::appendVHEATable):
(WebCore::CFFBuilder::CFFBuilder):
(WebCore::SVGToOTFFontConverter::transcodeGlyphPaths):
(WebCore::SVGToOTFFontConverter::processGlyphElement):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• svg/SVGToOTFFontConversion.cpp (modified) (6 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2015-01-19  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        [SVG -> OTF Converter] Glyphs get clipped weirdly
+        https://bugs.webkit.org/show_bug.cgi?id=137095
+
+        Reviewed by Antti Koivisto.
+
+        The Adobe CFF spec doesn't actually tell you how to serialize a
+        "FontBBox." After trial and error, it seems to be (x, y, width,
+        height).
+
+        Test: svg/text/kerning.svg
+              svg/W3C-SVG-1.1/fonts-kern-01-t.svg
+
+        * svg/SVGToOTFFontConversion.cpp:
+        (WebCore::SVGToOTFFontConverter::appendHEADTable):
+        (WebCore::SVGToOTFFontConverter::appendCFFTable):
+        (WebCore::SVGToOTFFontConverter::appendVHEATable):
+        (WebCore::CFFBuilder::CFFBuilder):
+        (WebCore::SVGToOTFFontConverter::transcodeGlyphPaths):
+        (WebCore::SVGToOTFFontConverter::processGlyphElement):
+
 2015-01-19  Csaba Osztrogonác  <ossy@webkit.org>
 

trunk/Source/WebCore/svg/SVGToOTFFontConversion.cpp

@@ -323 +323 @@
     append32(0); // First half of modification date
     append32(0); // Last half of modification date
-    append16(m_boundingBox.x()); // Minimum X
-    append16(m_boundingBox.y()); // Minimum Y
-    append16(m_boundingBox.maxX()); // Maximum X
-    append16(m_boundingBox.maxY()); // Maximum Y
+    append16(clampTo<int16_t>(m_boundingBox.x()));
+    append16(clampTo<int16_t>(m_boundingBox.y()));
+    append16(clampTo<int16_t>(m_boundingBox.maxX()));
+    append16(clampTo<int16_t>(m_boundingBox.maxY()));
     append16((m_italic ? 1 << 1 : 0) | (m_weight >= 7 ? 1 : 0));
     append16(3); // Smallest readable size in pixels
@@ -576 +576 @@
     append32(clampTo<int32_t>(m_boundingBox.x()));
     m_result.append(operand32Bit);
-    append32(clampTo<int32_t>(m_boundingBox.maxX()));
-    m_result.append(operand32Bit);
     append32(clampTo<int32_t>(m_boundingBox.y()));
     m_result.append(operand32Bit);
-    append32(clampTo<int32_t>(m_boundingBox.maxY()));
+    append32(clampTo<int32_t>(m_boundingBox.width()));
+    m_result.append(operand32Bit);
+    append32(clampTo<int32_t>(m_boundingBox.height()));
     m_result.append(fontBBoxKey);
     m_result.append(operand32Bit);
@@ -888 +888 @@
     append16(clampTo<int16_t>(-static_cast<int>(m_unitsPerEm / 2))); // Vertical typographic descender
     append16(m_unitsPerEm / 10); // Vertical typographic line gap
+    // FIXME: m_unitsPerEm is almost certainly not correct
     append16(clampTo<int16_t>(m_advanceHeightMax));
     append16(clampTo<int16_t>(m_unitsPerEm - m_boundingBox.maxY())); // Minimum top side bearing
@@ -1050 +1051 @@
         , m_hasBoundingBox(false)
     {
+        // FIXME: Moving to the origin isn't going to work for subsequent absolute coordinates
         writeCFFEncodedNumber(m_cffData, width);
         writeCFFEncodedNumber(m_cffData, origin.x());
@@ -1150 +1152 @@
     bool ok;
     float horizontalOriginX = glyphOrMissingGlyphElement.fastGetAttribute(SVGNames::horiz_origin_xAttr).toFloat(&ok);
-    if (!ok)
-        horizontalOriginX = m_fontFaceElement ? m_fontFaceElement->horizontalOriginX() : 0;
+    if (!ok && m_fontFaceElement)
+        horizontalOriginX = m_fontFaceElement->horizontalOriginX();
     float horizontalOriginY = glyphOrMissingGlyphElement.fastGetAttribute(SVGNames::horiz_origin_yAttr).toFloat(&ok);
     if (!ok && m_fontFaceElement)
-        horizontalOriginY = m_fontFaceElement ? m_fontFaceElement->horizontalOriginY() : 0;
+        horizontalOriginY = m_fontFaceElement->horizontalOriginY();
 
     CFFBuilder builder(result, width, FloatPoint(horizontalOriginX, horizontalOriginY));
@@ -1196 +1198 @@
     initialGlyph = false;
 
-    m_glyphs.append(GlyphData(WTF::move(path), glyphElement, horizontalAdvance, verticalAdvance, m_boundingBox, codepoints));
+    m_glyphs.append(GlyphData(WTF::move(path), glyphElement, horizontalAdvance, verticalAdvance, glyphBoundingBox, codepoints));
 }