Fix in vertical measurement in text lines with different text sizes

The last span of the TextLine was defining the FontMetrics, instead of
min/maxing the different values.

Bug 3220698

Change-Id: I7394b1699a15aeee4cc38462d561faf329d3e1f6

1 files changed, 20 insertions, 1 deletions

diff --git a/core/java/android/text/TextLine.java b/core/java/android/text/TextLine.java
index 2f7482c2f15d..a6fd2f11ab38 100644
--- a/core/java/android/text/TextLine.java
+++ b/core/java/android/text/TextLine.java
@@ -663,6 +663,25 @@ class TextLine {
     }
 
     /**
+     * @param wp
+     */
+    private static void expandMetricsFromPaint(FontMetricsInt fmi, TextPaint wp) {
+        final int previousTop     = fmi.top;
+        final int previousAscent  = fmi.ascent;
+        final int previousDescent = fmi.descent;
+        final int previousBottom  = fmi.bottom;
+        final int previousLeading = fmi.leading;
+
+        wp.getFontMetricsInt(fmi);
+
+        fmi.top     = Math.min(fmi.top,     previousTop);
+        fmi.ascent  = Math.min(fmi.ascent,  previousAscent);
+        fmi.descent = Math.max(fmi.descent, previousDescent);
+        fmi.bottom  = Math.max(fmi.bottom,  previousBottom);
+        fmi.leading = Math.max(fmi.leading, previousLeading);
+    }
+
+    /**
      * Utility function for measuring and rendering text.  The text must
      * not include a tab or emoji.
      *
@@ -703,7 +722,7 @@ class TextLine {
         }
 
         if (fmi != null) {
-            wp.getFontMetricsInt(fmi);
+            expandMetricsFromPaint(fmi, wp);
         }
 
         if (c != null) {