[Display Locking]: Highlight highest locked ancestor instead of highlighting nodes in locked subtree
Previously DevTools might call getBoundingClientRect on nodes in a
locked subtree from within paint, which might trigger style/layout
because we skipped the locked subtree in those phases (so they
might be still dirty), leading to crashes.
For nodes in a locked subtree, when we want to highlight it, we
should instead highlight the highest locked ancestor instead,
because it will always have up-to-date style & layout values.
The tooltip for the overlay will indicate that we're highlighting
the locked ancestor in that case.
for a screenshot of the UI.
Reviewed-by: Dmitry Gozman <email@example.com>
Reviewed-by: vmpstr <firstname.lastname@example.org>
Commit-Queue: Rakina Zata Amni <email@example.com>
13 files changed