Changeset 255227 in webkit

Timestamp:

Jan 27, 2020, 11:26:48 PM (5 years ago)

Author:

Antti Koivisto

Message:

Correct VTT Cue Style handling to match the specification
​https://bugs.webkit.org/show_bug.cgi?id=201086

Unreviewed followup based on a review comment.

• html/track/WebVTTParser.cpp:

(WebCore::WebVTTParser::checkAndStoreStyleSheet):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/track/WebVTTParser.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog ¶

@@ +1 @@
+2020-01-27  Antti Koivisto  <antti@apple.com>
+
+        Correct VTT Cue Style handling to match the specification
+        https://bugs.webkit.org/show_bug.cgi?id=201086
+
+        Unreviewed followup based on a review comment.
+
+        * html/track/WebVTTParser.cpp:
+        (WebCore::WebVTTParser::checkAndStoreStyleSheet):
+
 2020-01-27  Simon Fraser  <simon.fraser@apple.com>
 

trunk/Source/WebCore/html/track/WebVTTParser.cpp ¶

@@ -411 +411 @@
             continue;
 
-        sanitizedStyleSheetBuilder.append(selectorText);
-        sanitizedStyleSheetBuilder.appendLiteral(" { ");
-        sanitizedStyleSheetBuilder.append(styleRule.properties().asText());
-        sanitizedStyleSheetBuilder.appendLiteral(" }\n");
+        sanitizedStyleSheetBuilder.append(selectorText, " { ", styleRule.properties().asText(), "  }\n");
     }