Fixed invalid selector handling #98

FlorianRappl · FlorianRappl · commit 8b8482a41d89 · 2022-01-06T00:52:20.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 Released on Wednesday, January 5 2022.
 
 - Fixed issue with `text-shadow` missing the color part (#97)
+- Fixed support for `IsToleratingInvalidSelectors` parser option (#98)
 - Added `Color.UseHex` to change color output format (#96)
 
 # 0.16.2
diff --git a/src/AngleSharp.Css.Tests/Parsing/PseudoElements.cs b/src/AngleSharp.Css.Tests/Parsing/PseudoElements.cs
@@ -0,0 +1,33 @@
+namespace AngleSharp.Css.Tests.Parsing
+{
+    using AngleSharp.Css.Dom;
+    using AngleSharp.Css.Parser;
+    using AngleSharp.Html.Parser;
+    using NUnit.Framework;
+    using System.Linq;
+
+    [TestFixture]
+    public class PseudoElements
+    {
+        [Test]
+        public void VendorPrefixedPseudoElementShouldBeAvailable_Issue98()
+        {
+            var html = @"<html><head><style>
+    p::-webkit-scrollbar-thumb {
+        background: #888
+    }
+    </style></head><body><p>test</p></body></html>";
+            var parser = new HtmlParser(new HtmlParserOptions(), BrowsingContext.New(new Configuration().WithCss(new CssParserOptions
+            {
+                IsIncludingUnknownDeclarations = true,
+                IsIncludingUnknownRules = true,
+                IsToleratingInvalidSelectors = true,
+            })));
+            var document = parser.ParseDocument(html);
+            var stylesheet = document.StyleSheets.OfType<ICssStyleSheet>().First();
+            var rule = stylesheet.Rules.OfType<ICssStyleRule>().First();
+            var selectorText = rule.SelectorText;
+            Assert.AreEqual("p::-webkit-scrollbar-thumb", selectorText);
+        }
+    }
+}
diff --git a/src/AngleSharp.Css.Tests/Styling/CssCases.cs b/src/AngleSharp.Css.Tests/Styling/CssCases.cs
@@ -15,7 +15,7 @@ private static ICssStyleSheet ParseSheet(String text)
             {
                 IsIncludingUnknownDeclarations = true,
                 IsIncludingUnknownRules = true,
-                IsToleratingInvalidSelectors = true
+                IsToleratingInvalidSelectors = false,
             });
         }
 
diff --git a/src/AngleSharp.Css/Dom/Internal/Rules/CssPageRule.cs b/src/AngleSharp.Css/Dom/Internal/Rules/CssPageRule.cs
@@ -32,7 +32,7 @@ internal CssPageRule(ICssStyleSheet owner)
         public String SelectorText
         {
             get => _selector?.Text;
-            set { _selector = ParseSelector(value); ; }
+            set { _selector = ParseSelector(value); }
         }
 
         public ISelector Selector => _selector;
diff --git a/src/AngleSharp.Css/Dom/Internal/Rules/CssStyleRule.cs b/src/AngleSharp.Css/Dom/Internal/Rules/CssStyleRule.cs
@@ -1,5 +1,7 @@
 namespace AngleSharp.Css.Dom
 {
+    using AngleSharp.Css;
+    using AngleSharp.Dom;
     using System;
     using System.Diagnostics;
     using System.IO;
@@ -45,6 +47,11 @@ public String SelectorText
 
         #region Methods
 
+        internal void SetInvalidSelector(String selectorText)
+        {
+            _selector = new InvalidSelector(selectorText);
+        }
+
         protected override void ReplaceWith(ICssRule rule)
         {
             var newRule = (ICssStyleRule)rule;
@@ -59,5 +66,29 @@ public override void ToCss(TextWriter writer, IStyleFormatter formatter)
         }
 
         #endregion
-	}
+
+        #region Selector
+
+        class InvalidSelector : ISelector
+        {
+            private readonly String _text;
+
+            public InvalidSelector(String text)
+            {
+                _text = text;
+            }
+
+            public String Text => _text;
+
+            public Priority Specificity => Priority.Zero;
+
+            public void Accept(ISelectorVisitor visitor)
+            {
+            }
+
+            public Boolean Match(IElement element, IElement scope) => false;
+        }
+
+        #endregion
+    }
 }
diff --git a/src/AngleSharp.Css/Parser/CssBuilder.cs b/src/AngleSharp.Css/Parser/CssBuilder.cs
@@ -307,7 +307,14 @@ private CssSupportsRule CreateSupports(CssSupportsRule rule, CssToken current)
         public CssStyleRule CreateStyle(CssStyleRule rule, CssToken current)
         {
             CollectTrivia(ref current);
-            rule.SelectorText = GetArgument(ref current);
+            var selectorText = GetArgument(ref current);
+
+            rule.SelectorText = selectorText;
+
+            if (rule.Selector is null && _options.IsToleratingInvalidSelectors)
+            {
+                rule.SetInvalidSelector(selectorText);
+            }
 
             if (current.Type != CssTokenType.CurlyBracketOpen)
             {

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ private static ICssStyleSheet ParseSheet(String text)`
`15`	`15`	`{`
`16`	`16`	`IsIncludingUnknownDeclarations = true,`
`17`	`17`	`IsIncludingUnknownRules = true,`
`18`		`- IsToleratingInvalidSelectors = true`
	`18`	`+ IsToleratingInvalidSelectors = false,`
`19`	`19`	`});`
`20`	`20`	`}`
`21`	`21`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ internal CssPageRule(ICssStyleSheet owner)`
`32`	`32`	`public String SelectorText`
`33`	`33`	`{`
`34`	`34`	`get => _selector?.Text;`
`35`		`- set { _selector = ParseSelector(value); ; }`
	`35`	`+ set { _selector = ParseSelector(value); }`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`public ISelector Selector => _selector;`