Summary: | Update CSS Properties and Values API to use new cycle fallback behaviour | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Justin Michaud <justin_michaud> | ||||||||
Component: | Layout and Rendering | Assignee: | Justin Michaud <justin_michaud> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | bfulgham, commit-queue, dino, koivisto, rniwa, simon.fraser, webkit-bug-importer, zalan | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Justin Michaud
2018-12-17 21:26:55 PST
Created attachment 357532 [details]
Patch
Comment on attachment 357532 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=357532&action=review > Source/WebCore/css/CSSVariableReferenceValue.cpp:89 > + if ((!property || property->isInvalid()) && fallbackReturn) { > + result.appendVector(fallbackResult); > + return fallbackReturn; > + } > + > + if (!property || property->isInvalid()) > + return false; More compactly: if (!property || property->isInvalid()) { if (fallbackReturn) result.appendVector(fallbackResult); return fallbackReturn; } > LayoutTests/imported/w3c/web-platform-tests/css/css-properties-values-api/register-property-syntax-parsing-expected.txt:28 > -PASS syntax:'<length-percentage>', initialValue:'calc(-11px + 10.4%)' is valid > +FAIL syntax:'<length-percentage>', initialValue:'calc(-11px + 10.4%)' is valid The given initial value does not parse for the given syntax. Why do some of these go from PASS to FAIL? You could mention it in the ChangeLog. Created attachment 357578 [details]
Patch
Created attachment 357624 [details]
Patch
Comment on attachment 357624 [details] Patch Clearing flags on attachment: 357624 Committed r239365: <https://trac.webkit.org/changeset/239365> All reviewed patches have been landed. Closing bug. |