Closed Bug 1824957 Opened 2 years ago Closed 2 years ago

Stop using XUL layout for nsTreeBodyFrame.

Categories

(Core :: XUL, task)

task

Tracking

()

RESOLVED FIXED
113 Branch
Tracking Status
firefox113 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(3 files)

No description provided.

Surprisingly straight-forward:

Use SimpleXULLeafFrame, and override GetIntrinsicBSize to implement the rows
attribute.

Use DidReflow to manage the reflow callback which is equivalent to what the old
code did.

And some related tests.

A bunch of -moz-box* properties are web exposed, so I'll file a
follow-up for hiding them.

Depends on D173818

Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/30b054c6db5a Use SimpleXULLeafFrame for treebody. r=jwatt

The selection getter is nullable. Not having this check causes
browser_experimental_features_filter.js to fail due to the thrown
exception.

Pushed by nbeleuzu@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/103bb482b0bd Null-check tree selection in preferences search. r=Gijs
Blocks: 1825030

Backed out for causing reftest failures on treechildren-padding-percent-1.xhtml

Backout link

Push with failures

Failure log

Flags: needinfo?(emilio)
Attachment #9325410 - Attachment description: Bug 1824957 - Use SimpleXULLeafFrame for treebody. r=jwatt,#layout → Bug 1824957 - Use SimpleXULLeafFrame for treebody. r=jwatt
Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4afe60aa1ef4 Use SimpleXULLeafFrame for treebody. r=jwatt
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0e21add6bf2c Remove a bunch of now completely dead XUL layout code. r=layout-reviewers,desktop-theme-reviewers,devtools-reviewers,dao,TYLin
Keywords: leave-open
Regressions: 1825451
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch
See Also: → 1825489
Regressions: 1826881
Regressions: 1827580
No longer regressions: 1827580
Blocks: 1809492
Blocks: 1828283
Regressions: 1831535
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: