Closed Bug 1277890 Opened 4 years ago Closed 4 years ago

do less work with refcounting atoms when using mutation observers


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

Not set



Tracking Status
firefox50 --- fixed


(Reporter: froydnj, Assigned: froydnj)


(Whiteboard: btpp-active)


(3 files)

We have a lot of unnecessary refcounting going on.
This change is a quality-of-life API improvement, and there are a number of
other places where it can be used besides this bug.
Attachment #8759719 - Flags: review?(erahm)
We can pass the atom reference we have into the filter list, there's no
need for the filter list to retain a separate reference of the atom.
Attachment #8759720 - Flags: review?(amarchesini)
Once we have references to all the atoms for the filters list, we can
pass all those references in, rather than having to copy the array and
thus the references.
Attachment #8759721 - Flags: review?(amarchesini)
Whiteboard: btpp-active
Attachment #8759720 - Flags: review?(amarchesini) → review+
Attachment #8759721 - Flags: review?(amarchesini) → review+
Attachment #8759719 - Flags: review?(erahm) → review+
Pushed by
part 0 - add "append already_AddRefed element" API to nsCOMArray; r=erahm
part 1 - avoid unnecessary refcounting when building the filters list; r=baku
part 2 - avoid unnecessary refcounting when setting the filters list; r=baku
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.