Fonts forced to use non synthetic italics might be laid out with the incorrect baseline
[WebKit-https.git] / Source / WebCore / ChangeLog
index 5992a7a6bbfd9cac502651ba6ae71f12f72b3f4c..a1d687422140905bf062d5494da644d5b3fe8c62 100644 (file)
@@ -1,3 +1,27 @@
+2014-08-12  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        Fonts forced to use non synthetic italics might be laid out with the incorrect baseline
+        https://bugs.webkit.org/show_bug.cgi?id=135403
+
+        Reviewed by Darin Adler.
+
+        When italics is specified on a font, and the font isn't coming from the cache, we ask
+        the SimpleFontData to provide a non-synthetic-italic version of itself. Our current
+        implementation doesn't preserve whether or not the SimpleFontData includes vertical
+        glyphs (glyphs that are not rotated when drawn in the vertical writing mode), which
+        determines which baseline we use to lay out the text. By passing "false" to
+        the isTextOrientationFallback argument to SimpleFontData::create(), we preserve this
+        hasVerticalGlyphs flag.
+
+        Test: fast/text/international/synthesized-italic-vertical-latin-double.html
+
+        * platform/graphics/SimpleFontData.cpp:
+        (WebCore::SimpleFontData::nonSyntheticItalicFontData):
+        * testing/Internals.cpp:
+        (WebCore::Internals::invalidateFontCache): Add "invalidateFontCache" to window.internals.
+        * testing/Internals.h: Ditto.
+        * testing/Internals.idl: Ditto.
+
 2014-08-12  Peyton Randolph  <prandolph@apple.com>
 
         Runtime switch for long mouse press gesture. Part of 135257 - Add long mouse press gesture.