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
Avoid snapshot handling in animation only restyle #17770
Avoid snapshot handling in animation only restyle #17770
Conversation
In animation-only restyle, we just need to check the element has animation restyle hints or has recascade self which is a result of animation-only restyle for ancestors. has_current_styles() in Servo_ResolveStyle() is intentionally left there, it will be changed in a subsequent patch.
…ng throttle animations.
Heads up! This PR modifies the following files:
|
Waiting for being a try finished. |
8d6cf87
to
18cd739
Compare
The try looks good, but bindgen change causes weird errors [1], so I dropped Servo_StyleContext_AddRef and Servo_StyleContext_Release from the bindgen change. Thank you Xidorn for the workaround! https://pastebin.mozilla.org/9027416 @bors-servo r=emilio |
📌 Commit 18cd739 has been approved by |
…=emilio Avoid snapshot handling in animation only restyle <!-- Please describe your changes on the following line: --> https://bugzilla.mozilla.org/show_bug.cgi?id=1381431 --- <!-- 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 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17770) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev |
https://bugzilla.mozilla.org/show_bug.cgi?id=1381431
./mach build -d
does not report any errors./mach test-tidy
does not report any errorsThis change is