Open Bug 276857 Opened 20 years ago Updated 2 years ago

Tree column frame's mColumns can become stale

Categories

(Core :: XUL, defect)

defect

Tracking

()

People

(Reporter: neil, Unassigned)

Details

Attachments

(2 files)

If the tree body frame is destroyed (e.g. by removing the <treechildren> node
from the DOM and replacing it with a new one) then the tree column frame mColumn
caches become stale, and column changes no longer repaint automatically.
Attached patch Test CaseSplinter Review
Just try resizing, dragging or hiding any of the columns.
Attached patch Proposed patchSplinter Review
Attachment #170145 - Flags: review?(jan)
We could also iterate over tree col frames and clear the cached value or use
nsWeakReference. Anyway the patch looks good to me. r=varga
Attachment #170145 - Flags: review?(jan) → review+
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: xptoolkit.widgets
The plan is to remove the XUL "tree" widget in future (bug 1446335), but this has a patch, so for now I'm not going to wontfix this.
Assignee: jvarga → nobody
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: