Summary: | Re-use isCSSWideKeyword in CSSVariableParser.cpp & CSSPropertyParser.cpp | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tim Nguyen (:ntim) <ntim> | ||||
Component: | CSS | Assignee: | Tim Nguyen (:ntim) <ntim> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | darin, esprehn+autocc, ews-watchlist, glenn, gyuyoung.kim, koivisto, macpherson, menard, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 224816 | ||||||
Attachments: |
|
Description
Tim Nguyen (:ntim)
2021-11-08 12:02:51 PST
Created attachment 443584 [details]
Patch
Comment on attachment 443584 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=443584&action=review > Source/WebCore/css/parser/CSSPropertyParser.cpp:4568 > - CSSValueID id = m_range.peek().id(); > - if (id == CSSValueInherit || id == CSSValueInitial || id == CSSValueRevert) > + if (isCSSWideKeyword(m_range.peek().id())) Can we make a test for this? (In reply to Antti Koivisto from comment #2) > Comment on attachment 443584 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=443584&action=review > > > Source/WebCore/css/parser/CSSPropertyParser.cpp:4568 > > - CSSValueID id = m_range.peek().id(); > > - if (id == CSSValueInherit || id == CSSValueInitial || id == CSSValueRevert) > > + if (isCSSWideKeyword(m_range.peek().id())) > > Can we make a test for this? I don't see any easily extendable test in https://github.com/web-platform-tests/wpt/tree/master/css/css-properties-values-api unfortunately, or similar test for the matter. Committed r285483 (244007@main): <https://commits.webkit.org/244007@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 443584 [details]. Comment on attachment 443584 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=443584&action=review >>> Source/WebCore/css/parser/CSSPropertyParser.cpp:4568 >>> + if (isCSSWideKeyword(m_range.peek().id())) >> >> Can we make a test for this? > > I don't see any easily extendable test in https://github.com/web-platform-tests/wpt/tree/master/css/css-properties-values-api unfortunately, or similar test for the matter. Is this a behavior change, because we now check for "unset" as well? But not testable? |