-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Upgrade to Taffy v0.7.5 #34927
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade to Taffy v0.7.5 #34927
Conversation
|
🔨 Triggering try run (#12707003039) for Linux WPT |
mrobinson
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
taffyup!
|
I've removed this one from the queue to test whether or not the failure noted was a stable failure or not. |
|
🔨 Triggering try run (#12709264372) for Linux WPT |
|
🔨 Triggering try run (#12709272329) for Linux WPT |
|
Test results for linux-wpt-layout-2020 from try job (#12709264372): Flaky unexpected result (20)
Stable unexpected results that are known to be intermittent (6)
Stable unexpected results (1)
|
|
|
|
Test results for linux-wpt-layout-2020 from try job (#12709272329): Flaky unexpected result (14)
Stable unexpected results that are known to be intermittent (13)
Stable unexpected results (1)
|
|
|
|
🔨 Triggering try run (#12710072342) for Linux WPT |
|
Test results for linux-wpt-layout-2020 from try job (#12710072342): Flaky unexpected result (15)
Stable unexpected results that are known to be intermittent (16)
|
|
✨ Try run (#12710072342) succeeded. |
Signed-off-by: Nico Burns <nico@nicoburns.com>
Depends on: - servo/stylo#104 - #34926 - #34927 - #34948 In addition to that a `resolve_calc_value` function has been added which resolves calc values during the layout process once a percentage resolution basis is available. ~~There is 1 newly failing test and 1 newly failing subtest here.~~ These issues have now been fixed. There are 8 new subtest failures in `css/css-grid/grid-definition/grid-minimum-contribution-with-percentages.html`. These are genuine failures, but are unrelated to the calc implementation. The calc implementation is just exposing a pre-existing bug around percentage resolution that also now correctly also applies to `calc()` values containing percentages. The fix for would best be done in a followup as it requires teaching Taffy about "compressible replaced elements". (update: I have this [implemented](#34948) but in the interest of keeping it to one feature per PR I have not included it here) --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: --> - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors. It doesn't like the git dependency on Taffy. - [x] There are tests for these changes OR --------- Signed-off-by: Nico Burns <nico@nicoburns.com>
./mach build -ddoes not report any errors./mach test-tidydoes not report any errorsautokeyword is incorrect #34925