tree 5a1779d654096719923fb267af467b210d127692
parent acd7ef7bfdeef622367c4a3994a86e17885dd657
author Joey Arhar <jarhar@chromium.org> 1620937240 -0700
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1620938133 -0700

Fixed nested RecalcAssignment in DisplayLocking case

This loosens the flat tree traversal to a simple node traversal added by
crrev.com/877657, but I think that's OK since the scenarios of interest
in both that patch and this one are where the display lock and the slot
are in the same TreeScope. We can follow up on whether or not this is OK
later, I made a TODO for it.

The relaxed check, in combination with the removal of the HeapHashMap
and additional iteration over Slots(), likely fixed the performance bug
crbug.com/1204336 as well.

Fixed: 1208573, 1204336
Change-Id: I938cff05fa3acc9f65a9b0451acb1ad0b2ce6c56
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2891511
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#882664}