RESOLVED FIXED 232020
CSSPropertyZoom needs wrapper that ensures it's always blended into a positive value. 
https://bugs.webkit.org/show_bug.cgi?id=232020
Summary CSSPropertyZoom needs wrapper that ensures it's always blended into a positiv...
Gabriel Nava Marino
Reported 2021-10-20 10:25:01 PDT
It was recommended to consider "NonNegativeFloatPropertyWrapper as it could be refactored it to support various flags, for instance one for non-negative values, and one for strictly positive values, like we need for zoom"
Attachments
Patch (12.67 KB, patch)
2021-10-21 02:55 PDT, Antoine Quint 		koivisto: review+
ews-feeder: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gabriel Nava Marino
Comment 1 2021-10-20 10:56:06 PDT
<rdar://84469657>
Radar WebKit Bug Importer
Comment 2 2021-10-20 10:56:57 PDT
<rdar://problem/84469930>
Antoine Quint
Comment 3 2021-10-21 02:55:30 PDT
Created attachment 442001 [details] Patch
Antti Koivisto
Comment 4 2021-10-21 02:58:35 PDT
Comment on attachment 442001 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=442001&action=review > Source/WebCore/animation/CSSPropertyAnimation.cpp:2140 > + enum class ValueRange : uint8_t { > + All, > + NonNegative, > + Positive > + }; Nice
EWS
Comment 5 2021-10-21 04:53:38 PDT
Tools/Scripts/svn-apply failed to apply attachment 442001 [details] to trunk. Please resolve the conflicts and upload a new patch.
Antoine Quint
Comment 6 2021-10-21 05:40:19 PDT
Committed r284602 (243331@main): <https://commits.webkit.org/243331@main>
Note You need to log in before you can comment on or make changes to this bug.