Closed Bug 544503 Opened 15 years ago Closed 13 years ago

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

Categories

(Core :: DOM: Editor, defect)

Product:
Core
Component:
DOM: Editor
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: ehsan.akhgari)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fixed by bug 545775])

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
Closed: 13 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.