If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

QI'ing an nsGenericElement based element to nsIDOM* ends up in the binding manager code...

RESOLVED FIXED in mozilla0.9.9

Status

()

Core
DOM: Core & HTML
P2
normal
RESOLVED FIXED
16 years ago
9 years ago

People

(Reporter: jst, Assigned: jst)

Tracking

({perf})

Trunk
mozilla0.9.9
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [HAVE FIX])

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
When QI'ing an nsGenericElement based element to nsIDOM* we end up asking the
binding manager if there's a binding available that implements the nsIDOM*
interface before we check if the element itself implements that interface. Patch
coming up.
(Assignee)

Comment 1

16 years ago
Created attachment 65396 [details] [diff] [review]
Don't ask the binding manager if there's a binding that implements an interface before we know if the we implement it ourselves.
(Assignee)

Comment 2

16 years ago
Oh, this patch also fixes a potential range list leak.
Comment on attachment 65396 [details] [diff] [review]
Don't ask the binding manager if there's a binding that implements an interface before we know if the we implement it ourselves.

r/sr=brendan@mozilla.org
Attachment #65396 - Flags: review+
(Assignee)

Comment 4

16 years ago
Thanks brendan. hyatt, would you r= or sr=?
Status: NEW → ASSIGNED
Keywords: perf
Priority: -- → P2
Whiteboard: [HAVE FIX]
Target Milestone: --- → mozilla0.9.9

Updated

16 years ago
Attachment #65396 - Flags: superreview+
(Assignee)

Comment 5

16 years ago
Hyatt says r=hyatt.
(Assignee)

Comment 6

16 years ago
Fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Updated

9 years ago
Component: DOM: Core → DOM: Core & HTML
QA Contact: stummala → general
You need to log in before you can comment on or make changes to this bug.