Account for right bearing in QFontMetrics::boundingRect(string)

QFontMetrics::boundingRect() that takes a string needs to account for
the right bearing of the last glyph, as it is documented to be the
rectangle that contains the pixels of the text. I've added a test for
this, and fixed tst_QFontMetrics::elidedText() to use boundingRect() to
find the actual width of the text drawn (width() will return the advance
of the text, which is larger than the actual width of the pixels.)

I've also fixed a small typo in the "len" -> "ilen".

Reviewed-by: Simon Hausmann

1 files changed, 13 insertions, 1 deletions

diff --git a/tests/auto/qfontmetrics/tst_qfontmetrics.cpp b/tests/auto/qfontmetrics/tst_qfontmetrics.cpp
index ee6d442488..665107aacf 100644
--- a/tests/auto/qfontmetrics/tst_qfontmetrics.cpp
+++ b/tests/auto/qfontmetrics/tst_qfontmetrics.cpp
@@ -72,6 +72,7 @@ private slots:
     void veryNarrowElidedText();
     void averageCharWidth();
     void elidedMultiLength();
+    void bearingIncludedInBoundingRect();
 };
 
 tst_QFontMetrics::tst_QFontMetrics()
@@ -214,7 +215,7 @@ void tst_QFontMetrics::elidedMultiLength()
     QCOMPARE(fm.elidedText(text1,Qt::ElideRight, 8000), text1_long);
     QCOMPARE(fm.elidedText(text1,Qt::ElideRight, width_long + 1), text1_long);
     QCOMPARE(fm.elidedText(text1,Qt::ElideRight, width_long - 1), text1_short);
-    int width_short = fm.width(text1_short);
+    int width_short = fm.boundingRect(text1_short).width();
     QCOMPARE(fm.elidedText(text1,Qt::ElideRight, width_short + 1), text1_short);
     QCOMPARE(fm.elidedText(text1,Qt::ElideRight, width_short - 1), text1_small);
 
@@ -226,5 +227,16 @@ void tst_QFontMetrics::elidedMultiLength()
 
 }
 
+void tst_QFontMetrics::bearingIncludedInBoundingRect()
+{
+    QFont font;
+    font.setItalic(true);
+    QRect brectItalic = QFontMetrics(font).boundingRect("ITALIC");
+    font.setItalic(false);
+    QRect brectNormal = QFontMetrics(font).boundingRect("ITALIC");
+
+    QVERIFY(brectItalic.width() > brectNormal.width());
+}
+
 QTEST_MAIN(tst_QFontMetrics)
 #include "tst_qfontmetrics.moc"