Make nsMenuFrame::RemoveFrame not use DestroyFrameIfPresent

RESOLVED FIXED in mozilla22

Status

()

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: mats, Assigned: mats)

Tracking

Trunk
mozilla22
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
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))
(Assignee)

Comment 1

6 years ago
Created attachment 710778 [details] [diff] [review]
fix
Attachment #710778 - Flags: review?(bzbarsky)
(Assignee)

Updated

6 years ago
Assignee: nobody → matspal
(Assignee)

Updated

6 years ago
Blocks: 838706
Comment on attachment 710778 [details] [diff] [review]
fix

r=me
Attachment #710778 - Flags: review?(bzbarsky) → review+

Comment 4

6 years ago
https://hg.mozilla.org/mozilla-central/rev/aee03685cd2b
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.