Bug 118260 – REGRESSION(r151510): CSS parser recovery for extra left parenthesis causes ignoring definitions beyond

• Status: RESOLVED
• Resolution: WONTFIX
• Component: CSS

[alan baradlay]

with the following CSS

.foo {
    background: rgba(255,255,255,1.0)
	    -moz-linear-gradient(top,
	    rgba(170,175,178,0.8) 15%,
	    (rgba(255,255,255,0.5) 65%
	);
}

div { 
    border: 1px solid red;
}

the extra '(' right before the second rgb causes ignoring the div { border: 1px.....

[alan baradlay]

Created attachment 205828 [details]
test case

[alan baradlay]

<rdar://problem/14321487>

[alan baradlay]

regressed at http://trac.webkit.org/changeset/151510

[Sergio Villar Senin]

Taking a look...

[Sergio Villar Senin]

Actually I'm not sure this is a regression. In the "Rules for handling parsing errors" section of the CSS specs it mentions:

"User agents must close all open constructs (for example: blocks, parentheses, brackets, rules, strings, and comments) at the end of the style sheet."

So I understand from this paragraph that the new behavior is the desired one (actually no browser supports the test case you attached as all of them look for the matching closing parenthesis till the end of the stylesheet). Adding some more experienced people so we could have another opinion.

[Ahmad Saleem]

I am unable to reproduce this bug in Safari Technology Preview 151 and it does not show "red" border and it is same across all other browsers (Chrome Canary 106 and Firefox Nightly 105). I am going to mark this as "RESOLVED WONTFIX" since all browsers are aligned.

Please reopen if you think this is still not correct intended beahvior or I am testing incorrectly. Thanks!