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

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P2)

Description

[Johnny Stenback (:jst)]

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.

Comment 1

[Johnny Stenback (:jst)]

Attachment: Don't ask the binding manager if there's a binding that implements an interface before we know if the we implement it ourselves.

Comment 2

[Johnny Stenback (:jst)]

Oh, this patch also fixes a potential range list leak.

Comment 3

[Brendan Eich [:brendan]]

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=[e-mail]

Comment 4

[Johnny Stenback (:jst)]

Thanks brendan. hyatt, would you r= or sr=?

Comment 5

[Johnny Stenback (:jst)]

Hyatt says r=hyatt.

Comment 6

[Johnny Stenback (:jst)]

Fixed.