Mirrors/uBlock
1
0
Fork 0
mirror of https://github.com/gorhill/uBlock.git synced 2024-09-15 07:22:28 +02:00
Code Issues Releases Wiki Activity

Revert "Better detect invalid cosmetic filters"

Browse Source 
This reverts commit a8ca5127c9.
This commit is contained in:
Raymond Hill 2022-07-15 21:49:05 -04:00
parent d82626d6cb
commit 19298fecf3
No known key found for this signature in database
GPG Key ID: 25E1490B761470C2
1 changed files with 29 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

55
src/js/static-filtering-parser.js
View File

@ -19,7 +19,7 @@
Home: https://github.com/gorhill/uBlock Home: https://github.com/gorhill/uBlock
*/ */
/* globals document */ /* globals CSSStyleSheet, document */
'use strict'; 'use strict';
@ -1333,17 +1333,20 @@ Parser.prototype.SelectorCompiler = class {
[ 'matches-css-before', ':matches-css-before' ], [ 'matches-css-before', ':matches-css-before' ],
]); ]);
this.reSimpleSelector = /^[#.]?[A-Za-z_][\w-]*$/; this.reSimpleSelector = /^[#.]?[A-Za-z_][\w-]*$/;
this.styleElement = (( ) => { // https://developer.mozilla.org/en-US/docs/Web/API/CSSStyleSheet#browser_compatibility
// Firefox does not support constructor for CSSStyleSheet
this.stylesheet = (( ) => {
if ( typeof document !== 'object' ) { return null; } if ( typeof document !== 'object' ) { return null; }
if ( document === null ) { return null; } if ( document instanceof Object === false ) { return null; }
try { try {
const styleElement = document.createElement('style'); return new CSSStyleSheet();
styleElement.disabled = true;
document.body.append(styleElement);
return styleElement;
} catch(ex) { } catch(ex) {
} }
return null; const style = document.createElement('style');
document.body.append(style);
const stylesheet = style.sheet;
style.remove();
return stylesheet;
})(); })();
this.div = (( ) => { this.div = (( ) => {
if ( typeof document !== 'object' ) { return null; } if ( typeof document !== 'object' ) { return null; }
@ -1465,17 +1468,18 @@ Parser.prototype.SelectorCompiler = class {
// Forbid multiple and unexpected CSS style declarations. // Forbid multiple and unexpected CSS style declarations.
sheetSelectable(s) { sheetSelectable(s) {
if ( this.reSimpleSelector.test(s) ) { return true; } if ( this.reSimpleSelector.test(s) ) { return true; }
if ( this.styleElement === null ) { return true; } if ( this.stylesheet === null ) { return true; }
try { try {
this.styleElement.textContent = `${s}{color:red}`; this.stylesheet.insertRule(`${s}{color:red}`);
const rules = this.styleElement.sheet.cssRules; if ( this.stylesheet.cssRules.length !== 1 ) { return false; }
if ( rules.length !== 1 ) { return false; } const style = this.stylesheet.cssRules[0].style;
const style = rules[0].style;
if ( style.length !== 1 ) { return false; } if ( style.length !== 1 ) { return false; }
if ( style.getPropertyValue('color') === 'red' ) { return true; } if ( style.getPropertyValue('color') !== 'red' ) { return false; }
this.stylesheet.deleteRule(0);
} catch (ex) { } catch (ex) {
return false;
} }
return false; return true;
} }
// https://github.com/uBlockOrigin/uBlock-issues/issues/1806 // https://github.com/uBlockOrigin/uBlock-issues/issues/1806
@ -1596,15 +1600,17 @@ Parser.prototype.SelectorCompiler = class {
// - opening comment `/*` // - opening comment `/*`
compileStyleProperties(s) { compileStyleProperties(s) {
if ( /image-set\(|url\(|\/\s*\/|\\|\/\*/i.test(s) ) { return; } if ( /image-set\(|url\(|\/\s*\/|\\|\/\*/i.test(s) ) { return; }
if ( this.styleElement === null ) { return s; } if ( this.stylesheet === null ) { return s; }
let valid = false; let valid = false;
try { try {
this.styleElement.textContent = `a{${s}} b{color:red;}`; this.stylesheet.insertRule(`a{${s}}`);
const rules = this.styleElement.sheet.cssRules; const rules = this.stylesheet.cssRules;
valid = rules.length >= 2 && valid = rules.length !== 0 && rules[0].style.cssText !== '';
rules[0].style.cssText !== '' &&
rules[1].style.cssText !== '';
} catch(ex) { } catch(ex) {
return;
}
if ( this.stylesheet.cssRules.length !== 0 ) {
this.stylesheet.deleteRule(0);
} }
if ( valid ) { return s; } if ( valid ) { return s; }
} }
@ -1775,12 +1781,8 @@ Parser.prototype.SelectorCompiler = class {
if ( i === n ) { break; } if ( i === n ) { break; }
} }
// when there is an explicit action, nothing should be left to parse.
if ( action !== undefined && opPrefixBeg < n ) { return; }
// No task found: then we have a CSS selector. // No task found: then we have a CSS selector.
// At least one task found: either nothing or a valid plain CSS selector // At least one task found: nothing should be left to parse.
// should be left to parse
if ( tasks.length === 0 ) { if ( tasks.length === 0 ) {
if ( action === undefined ) { if ( action === undefined ) {
prefix = raw; prefix = raw;
@ -1794,6 +1796,7 @@ Parser.prototype.SelectorCompiler = class {
} }
} else if ( opPrefixBeg < n ) { } else if ( opPrefixBeg < n ) {
if ( action !== undefined ) { return; }
const spath = this.compileSpathExpression(raw.slice(opPrefixBeg)); const spath = this.compileSpathExpression(raw.slice(opPrefixBeg));
if ( spath === undefined ) { return; } if ( spath === undefined ) { return; }
tasks.push([ ':spath', spath ]); tasks.push([ ':spath', spath ]);