612159 - Switch to passing nsINode containers in nsIMutationObserver methods

Status: NEW

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

Description

[(no longer active)]

The container we pass to nsIMutationObserver methods can either be a document or a content node.  We currently pass an nsIContent pointer to such methods, which makes the kind of dance in attachment 490417 [details] [diff] [review] necessary (because aContainer will be null if the container is a document).

The saner approach would be to pass an nsINode pointer to these methods.

Boris, is nsIMutationObserver a public interface which we can't change before we branch?

Comment 1

[Boris Zbarsky [:bzbarsky]]

Yes.  I also think this shoudn't be happening before 2.0, since I suspect the patch will be somewhat large.

Comment 2

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.