Closed Bug 838679 Opened 9 years ago Closed 9 years ago

Make nsMenuFrame::RemoveFrame not use DestroyFrameIfPresent

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: mats, Assigned: mats)

References

Details

Attachments

(1 file)

The popupList only has one frame on it so we can just the first child.
DestroyPopupList() asserts it's empty.

(the goal is to remove DestroyFrameIfPresent which is O(n))
Attached patch fixSplinter Review
Attachment #710778 - Flags: review?(bzbarsky)
Assignee: nobody → matspal
Blocks: 838706
Comment on attachment 710778 [details] [diff] [review]
fix

r=me
Attachment #710778 - Flags: review?(bzbarsky) → review+
https://hg.mozilla.org/mozilla-central/rev/aee03685cd2b
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.