mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-09-20 08:18:55 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibWeb: Don't consider null -> null as a change for invalidation
Some checks are pending
CI / macOS, arm64, Sanitizer, Clang (push) Waiting to run
Details
CI / Linux, x86_64, Fuzzers, Clang (push) Waiting to run
Details
CI / Linux, x86_64, Sanitizer, GNU (push) Waiting to run
Details
CI / Linux, x86_64, Sanitizer, Clang (push) Waiting to run
Details
Package the js repl as a binary artifact / Linux, arm64 (push) Waiting to run
Details
Package the js repl as a binary artifact / macOS, arm64 (push) Waiting to run
Details
Package the js repl as a binary artifact / Linux, x86_64 (push) Waiting to run
Details
Run test262 and test-wasm / run_and_update_results (push) Waiting to run
Details
Lint Code / lint (push) Waiting to run
Details
Label PRs with merge conflicts / auto-labeler (push) Waiting to run
Details
Push notes / build (push) Waiting to run
Details

Browse source 
This avoids excessive invalidation within `recompute_inherited_style`
where we inherit a value but not an animated value.
This commit is contained in:
Callum Law 2025-08-22 16:43:56 +12:00 • committed by Jelle Raaijmakers
commit 5c2fe12772
Notes: github-actions[bot] 2025-08-22 10:19:01 +00:00
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Libraries/LibWeb/CSS/StyleInvalidation.cpp
View file

@ -15,7 +15,7 @@ RequiredInvalidationAfterStyleChange compute_property_invalidation(CSS::Property
{
RequiredInvalidationAfterStyleChange invalidation;
bool const property_value_changed = (!old_value || !new_value) || *old_value != *new_value;
bool const property_value_changed = (old_value || new_value) && ((!old_value || !new_value) || *old_value != *new_value);
if (!property_value_changed)
return invalidation;