Attachment 361959 Details for Bug 194227 – Patch

[patch] Patch

bug-194227-20190213163219.patch (text/plain), 4.20 KB, created by Per Arne Vollan on 2019-02-13 16:32:20 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Per Arne Vollan

Created: 2019-02-13 16:32:20 PST

Size: 4.20 KB

patch

obsolete

>Index: Source/WebCore/ChangeLog
>===================================================================
>--- Source/WebCore/ChangeLog	(revision 241484)
>+++ Source/WebCore/ChangeLog	(working copy)
>@@ -1,3 +1,19 @@
>+2019-02-13  Per Arne Vollan  <pvollan@apple.com>
>+
>+        [WebVTT] Inline WebVTT styles should start with '::cue'
>+        https://bugs.webkit.org/show_bug.cgi?id=194227
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        The original fix in r241203 is not sufficient, since it only checks if the CSS string starts
>+        with '::cue'. Before accepting a CSS string from a WebVTT file, it should be checked that
>+        all selectors starts with '::cue'.
>+
>+        Test: media/track/track-cue-css.html
>+
>+        * html/track/WebVTTParser.cpp:
>+        (WebCore::WebVTTParser::checkAndStoreStyleSheet):
>+
> 2019-02-13  Jer Noble  <jer.noble@apple.com>
> 
>         Entering fullscreen inside a shadow root will not set fullscreen pseudoclasses outside of root
>Index: Source/WebCore/html/track/WebVTTParser.cpp
>===================================================================
>--- Source/WebCore/html/track/WebVTTParser.cpp	(revision 241445)
>+++ Source/WebCore/html/track/WebVTTParser.cpp	(working copy)
>@@ -39,6 +39,8 @@
> #include "HTMLParserIdioms.h"
> #include "ISOVTTCue.h"
> #include "ProcessingInstruction.h"
>+#include "StyleRule.h"
>+#include "StyleRuleImport.h"
> #include "StyleSheetContents.h"
> #include "Text.h"
> #include "VTTScanner.h"
>@@ -369,21 +371,38 @@ bool WebVTTParser::checkAndStoreStyleShe
>     if (!line.isEmpty() && !line.contains("-->"))
>         return false;
>     
>-    auto styleSheet = m_currentStyleSheet.stripWhiteSpace();
>+    auto styleSheet = WTFMove(m_currentStyleSheet);
>     
>-    // Inline VTT styles must start with ::cue.
>-    if (!styleSheet.startsWith("::cue")) {
>-        m_currentStyleSheet = emptyString();
>+    auto contents = StyleSheetContents::create();
>+    if (!contents->parseString(styleSheet))
>         return true;
>-    }
> 
>-    auto contents = StyleSheetContents::create();
>-    if (!contents->parseString(styleSheet)) {
>-        m_currentStyleSheet = emptyString();
>+    auto namespaceRules = contents->namespaceRules();
>+    if (namespaceRules.size())
>+        return true;
>+
>+    auto importRules = contents->importRules();
>+    if (importRules.size())
>         return true;
>+
>+    auto childRules = contents->childRules();
>+    for (auto rule : childRules) {
>+        if (!rule->isStyleRule())
>+            return true;
>+        const auto& styleRule = downcast<StyleRule>(rule.get());
>+
>+        const auto& selectorList = styleRule->selectorList();
>+        auto listSize = selectorList.listSize();
>+        if (!listSize)
>+            return true;
>+        for (unsigned i = 0; i < listSize; i++) {
>+            auto selector = selectorList.selectorAt(i);
>+            if (selector->selectorText() != "::cue")
>+                return true;
>+        }
>     }
>-    
>-    m_styleSheets.append(WTFMove(m_currentStyleSheet));
>+
>+    m_styleSheets.append(styleSheet);
>     return true;
> }
> 
>Index: LayoutTests/ChangeLog
>===================================================================
>--- LayoutTests/ChangeLog	(revision 241445)
>+++ LayoutTests/ChangeLog	(working copy)
>@@ -1,3 +1,14 @@
>+2019-02-13  Per Arne Vollan  <pvollan@apple.com>
>+
>+        [WebVTT] Inline WebVTT styles should start with '::cue'
>+        https://bugs.webkit.org/show_bug.cgi?id=194227
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add 'STYLE' block with invalid selector 'video::cue'.
>+
>+        * media/track/captions-webvtt/css-styling.vtt:
>+
> 2019-02-13  Sihui Liu  <sihui_liu@apple.com>
> 
>         Add two regression tests for reference cycle in IndexedDB
>Index: LayoutTests/media/track/captions-webvtt/css-styling.vtt
>===================================================================
>--- LayoutTests/media/track/captions-webvtt/css-styling.vtt	(revision 241445)
>+++ LayoutTests/media/track/captions-webvtt/css-styling.vtt	(working copy)
>@@ -31,6 +31,18 @@ color: blue;
> font-size: 25px;
> }
> 
>+NOTE the following style block should be discarded since it has a 'video::cue' selector.
>+
>+STYLE
>+::cue {
>+color: blue
>+font-size: 25px;
>+}
>+video::cue {
>+color: blue;
>+font-size: 25px;
>+}
>+
> hello
> 00:00:00.000 --> 00:00:10.000
> <b>Hello</b> first cue.

Actions: View | Formatted Diff | Diff

Attachments on bug 194227: 361081 | 361083 | 361514 | 361516 | 361956 | 361959 | 362030 | 362045 | 362069 | 362120