Closed Bug 1825030 Opened 1 year ago Closed 1 year ago

Figure out why D173847 is needed.

Categories

(Core :: XUL, defect)

defect

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: emilio, Assigned: emilio)

References

Details

It seems https://phabricator.services.mozilla.com/D173847 is needed because before we were hitting the "view is null" case and now the view is not null but the selection is.

We should understand that subtle behavior change just in case it can cause deeper issues (though I doubt it'll be an issue in practice, since bug 1802533 shows what happens r/n if you reframe trees).

Flags: needinfo?(emilio)

So this is just about us flushing layout at different times. XULTreeElement.view has always had a pretty weird caching story. I don't think it is worth emulating the old behavior.

Status: NEW → RESOLVED
Closed: 1 year ago
Flags: needinfo?(emilio)
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.