Switch SubtitleWebView to use single-quotes in generated HTML

The current state is inconsistent with SpannedToHtmlConverter, it seems they might as well agree on quoting strategy. PiperOrigin-RevId: 310926683

Switch SubtitleWebView to use single-quotes in generated HTML
The current state is inconsistent with SpannedToHtmlConverter, it seems they might as well agree on quoting strategy. PiperOrigin-RevId: 310926683
0fb4ba78 · ibaker · Oliver Woodman · 8528129e · 0fb4ba78
Commit 0fb4ba78 authored May 11, 2020 by ibaker Committed by Oliver Woodman May 14, 2020
Showing with 5 additions and 5 deletions
library/ui/src/main/java/com/google/android/exoplayer2/ui/SubtitleWebView.java
--- a/library/ui/src/main/java/com/google/android/exoplayer2/ui/SubtitleWebView.java
+++ b/library/ui/src/main/java/com/google/android/exoplayer2/ui/SubtitleWebView.java
@@ -127,7 +127,7 @@ import java.util.List;
    StringBuilder html = new StringBuilder();
    html.append(
        Util.formatInvariant(
-            "<html><body><div style=\""
+            "<html><body><div style='"
                + "-webkit-user-select:none;"
                + "position:fixed;"
                + "top:0;"
@@ -136,7 +136,7 @@ import java.util.List;
                + "right:0;"
                + "color:%s;"
                + "font-size:%s;"
-                + "\">",
+                + "'>",
            HtmlUtils.toCssRgba(style.foregroundColor),
            convertTextSizeToCss(defaultTextSizeType, defaultTextSize)));

@@ -222,7 +222,7 @@ import java.util.List;

      html.append(
              Util.formatInvariant(
-                  "<div style=\""
+                  "<div style='"
                      + "position:absolute;"
                      + "%s:%.2f%%;"
                      + "%s:%.2f%%;"
@@ -232,7 +232,7 @@ import java.util.List;
                      + "font-size:%s;"
                      + "background-color:%s;"
                      + "transform:translate(%s%%,%s%%);"
-                      + "\">",
+                      + "'>",
                  positionProperty,
                  positionPercent,
                  lineProperty,
@@ -245,7 +245,7 @@ import java.util.List;
                  windowCssColor,
                  horizontalTranslatePercent,
                  verticalTranslatePercent))
-          .append(Util.formatInvariant("<span style=\"background-color:%s;\">", backgroundColorCss))
+          .append(Util.formatInvariant("<span style='background-color:%s;'>", backgroundColorCss))
          .append(
              SpannedToHtmlConverter.convert(
                  cue.text, getContext().getResources().getDisplayMetrics().density))