Editor should not observe all mutations on the document if it's initialized with a root node

RESOLVED FIXED

Status

()

Core
Editor
RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: bz, Assigned: Ehsan)

Tracking

(Blocks: 1 bug)

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [fixed by bug 545775])

See bug 543494 comment 10.
Blocks: 543494
OK.  This mutation observer was added in bug 335856.  It looks to me like this code is wholly unnecessary for the case when the editor is initialized with a given root.  Peter, is that correct?  If not, what parts of it are needed?  Certainly in the text control case this code never gets triggered in a useful way!
Ehsan said he can take this, but Peter's feedback here would still be really nice.
Assignee: nobody → ehsan.akhgari
Yeah, afaics that should be fine.
So bug 545775 part 4 moved the observers from nsEditor to nsHTMLEditor.  Is this bug still relevant?
(In reply to Aryeh Gregor from comment #4)
> So bug 545775 part 4 moved the observers from nsEditor to nsHTMLEditor.  Is
> this bug still relevant?

No!  :-)
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [fixed by bug 545775]
You need to log in before you can comment on or make changes to this bug.