Merge pull request #8814 from dlafayet:line-height

PiperOrigin-RevId: 368803206

RELEASENOTES.md

@@ -77,6 +77,8 @@
 *   DASH:
 *   DASH:
     *   Parse `forced_subtitle` role from DASH manifests
     *   Parse `forced_subtitle` role from DASH manifests
         ([#8781](https://github.com/google/ExoPlayer/issues/8781)).
         ([#8781](https://github.com/google/ExoPlayer/issues/8781)).
+*   Text:
+    *   Fix lines overlapping when using `SubtitleView.VIEW_TYPE_WEB`.
 *   MediaSession extension: Remove dependency to core module and rely on common
 *   MediaSession extension: Remove dependency to core module and rely on common
     only. The `TimelineQueueEditor` uses a new `MediaDescriptionConverter` for
     only. The `TimelineQueueEditor` uses a new `MediaDescriptionConverter` for
     this purpose and does not rely on the `ConcatenatingMediaSource` anymore.
     this purpose and does not rely on the `ConcatenatingMediaSource` anymore.

library/ui/src/main/java/com/google/android/exoplayer2/ui/WebViewSubtitleOutput.java

@@ -175,7 +175,7 @@ import java.util.Map;
                 + "right:0;"
                 + "right:0;"
                 + "color:%s;"
                 + "color:%s;"
                 + "font-size:%s;"
                 + "font-size:%s;"
-                + "line-height:%.2fem;"
+                + "line-height:%.2f;"
                 + "text-shadow:%s;"
                 + "text-shadow:%s;"
                 + "'>",
                 + "'>",
             HtmlUtils.toCssRgba(style.foregroundColor),
             HtmlUtils.toCssRgba(style.foregroundColor),