EditorBase::EndPlaceholderTransaction() shows up in performance profiles

NEW
Unassigned

Status

()

Core
Editor
10 days ago
9 days ago

People

(Reporter: smaug, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

10 days ago
I wonder if we could have some fast paths there.
EditorBase::EndPlaceholderTransaction() and stuff under it take currently 12% of input.value setting.
Which method called from EndPlaceholderTransaction() is slow? If EditorBase::NotifyEditorObservers() is slow, we could make it faster with reducing nsIEditorObserver instances (we could make treat some of them as concrete classes).
(Reporter)

Comment 2

9 days ago
It was many thing. I think I was profiling using Zoom, so sharing the profile isn't quite as easy as with Gecko profiler. I used Test 2 from bug 1346723 for testing.
You need to log in before you can comment on or make changes to this bug.