[Qt] Monospace font does not render in proper sizes
authorallan.jensen@digia.com <allan.jensen@digia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Mar 2013 14:28:39 +0000 (14:28 +0000)
committerallan.jensen@digia.com <allan.jensen@digia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Mar 2013 14:28:39 +0000 (14:28 +0000)
https://bugs.webkit.org/show_bug.cgi?id=93263

Reviewed by Jocelyn Turcotte.

Do not force fonts to use integer metrics. This was necessary when the fast font path
only worked in integer but is now hindering accurate metrics. This is also required to
make kerning in the fast-path look acceptable

Only enabled for Qt 5.1+ to avoid changing all the baselines for 5.0.

* platform/graphics/qt/FontPlatformDataQt.cpp:
(WebCore::FontPlatformData::FontPlatformData):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@146203 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/qt/FontPlatformDataQt.cpp

index 0ba9c9b..6dff837 100644 (file)
@@ -1,3 +1,19 @@
+2013-03-19  Allan Sandfeld Jensen  <allan.jensen@digia.com>
+
+        [Qt] Monospace font does not render in proper sizes
+        https://bugs.webkit.org/show_bug.cgi?id=93263
+
+        Reviewed by Jocelyn Turcotte.
+
+        Do not force fonts to use integer metrics. This was necessary when the fast font path
+        only worked in integer but is now hindering accurate metrics. This is also required to
+        make kerning in the fast-path look acceptable
+
+        Only enabled for Qt 5.1+ to avoid changing all the baselines for 5.0.
+
+        * platform/graphics/qt/FontPlatformDataQt.cpp:
+        (WebCore::FontPlatformData::FontPlatformData):
+
 2013-03-19  John J. Barton  <johnjbarton@chromium.org>
 
         Web Inspector: Add iframe option to inspectedWindow.eval() extension API
index 529150a..78d3c3c 100644 (file)
@@ -71,7 +71,10 @@ FontPlatformData::FontPlatformData(const FontDescription& description, const Ato
     font.setWeight(toQFontWeight(description.weight()));
     font.setWordSpacing(wordSpacing);
     font.setLetterSpacing(QFont::AbsoluteSpacing, letterSpacing);
+#if QT_VERSION < QT_VERSION_CHECK(5, 1, 0)
+    // Kept enabled for Qt < 5.1 to maintain stable baselines for 5.0.
     font.setStyleStrategy(QFont::ForceIntegerMetrics);
+#endif
 
     m_data->bold = font.bold();
     // WebKit allows font size zero but QFont does not. We will return