Summary: | A number of offscreencanvas tests are asserting isMainThread trying to access strictCSSParserContext | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Lauro Moura <lmoura> | ||||
Component: | Canvas | Assignee: | Fujii Hironori <Hironori.Fujii> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bugs-noreply, cgarcia, clord, dino, esprehn+autocc, ews-watchlist, glenn, gyuyoung.kim, Hironori.Fujii, macpherson, menard, sam, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=221881 | ||||||
Attachments: |
|
Description
Lauro Moura
2021-02-18 20:18:33 PST
Gardened in r273124. About the mentioned revisions, they are the last good and first bad on the bots. A candidate in the range seems to be r272870/bug221881 "Prepare for adding relative color support". Cc'ing Sam. Created attachment 420929 [details]
Patch
Comment on attachment 420929 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=420929&action=review > Source/WebCore/css/parser/CSSParser.cpp:120 > - return CSSPropertyParserHelpers::consumeColorWorkerSafe(range, strictCSSParserContext()); > + return CSSPropertyParserHelpers::consumeColorWorkerSafe(range, CSSParserContext(HTMLStandardMode)); Why is this GLib specific? Comment on attachment 420929 [details]
Patch
FWIW, this looks good to me.
(In reply to Carlos Garcia Campos from comment #3) > Comment on attachment 420929 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=420929&action=review > > > Source/WebCore/css/parser/CSSParser.cpp:120 > > - return CSSPropertyParserHelpers::consumeColorWorkerSafe(range, strictCSSParserContext()); > > + return CSSPropertyParserHelpers::consumeColorWorkerSafe(range, CSSParserContext(HTMLStandardMode)); > > Why is this GLib specific? OffscreenCanvas is only enabled on GLib, this function is only used there. Committed r273156 (234351@main): <https://commits.webkit.org/234351@main> |