Bug 253462

Summary: CSS @imports in HTML missing semi-colon and space are mistakenly hidden from CSSPreload Scanner
Product: WebKit Reporter: Ahmad Saleem <ahmad.saleem792>
Component: Page LoadingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: beidson, karlcow, koivisto, rniwa, webkit-bug-importer
Priority: P2 Keywords: BrowserCompat, InRadar, WPTImpact
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

Description Ahmad Saleem 2023-03-06 13:56:33 PST
Hi Team,

This is continuation bug from bug 191466, where merging Chromium / Blink patch has now resolved one case from our CSS Preloader where 'quote-marks' work.

Attaching Chromium / Blink patch resulted in failing following testcase:

http/tests/loading/preload-css-import-layer.html

______

Antti suggested that it could be because of following:

I think this is wrong since there may be more things to parse after the first ) like in

url("foo.css") layer(foo)

This is causing the test failure.

_____

WPT Test Case - https://wpt.fyi/results/loading?label=master&label=experimental&aligned=&view=subtest&q=loading%2F

preloader-css-import-no-space.tentative.html & preloader-css-import-no-semicolon.tentative.html

_____

Following was skipped from Chromium / Blink patch:

// When reading the rule and hitting ')', which signifies the URL end, emit the rule.
            if (c == ')')
                emitRule();  

While this patch seems to suggest 'Chromium' support of layers in CSS Preloader:

https://chromium.googlesource.com/chromium/src/+/d53bf2d989eb0756ae2689f1b16c2875e5384e5e

_________

Just wanted to raise this so we can track failing test cases and fix them since all other browsers are passing them.

Thanks!
Comment 1 Radar WebKit Bug Importer 2023-03-13 14:57:15 PDT
<rdar://problem/106666050>
Comment 2 Ahmad Saleem 2023-08-24 16:09:23 PDT
I have local patch, which progress these two tests but don't know if it cause any other regression, I will do draft PR to test and document (if I couldn't manage to land for any reason).
Comment 3 EWS 2023-08-26 03:12:20 PDT
Committed 267322@main (af134e645543): <https://commits.webkit.org/267322@main>

Reviewed commits have been landed. Closing PR #17043 and removing active labels.