Closed Bug 807465 Opened 12 years ago Closed 12 years ago

Less QIs in nsXULPopupManager::UpdateMenuItems

Categories

(Core :: XUL, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla19

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Details

Attachments

(1 file)

Attached patch Patch v1 (deleted) — Splinter Review
There is no point in using nsIDOM* interfaces here.
Attachment #677156 - Flags: review?
Attachment #677156 - Flags: review? → review?(enndeakin)
Comment on attachment 677156 [details] [diff] [review] Patch v1 >+ nsRefPtr<dom::Element> commandElt = document->GetElementById(command); >+ if (commandElement) { You meant to assign to 'commandElement' not 'commandElt'.
Yes, I did. Looks like I forgot to refresh my patch after fixing that.
Comment on attachment 677156 [details] [diff] [review] Patch v1 This is ok then assuming that change gets made.
Attachment #677156 - Flags: review?(enndeakin) → review+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Component: XP Toolkit/Widgets: XUL → XUL
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: