Closed Bug 599074 Opened 14 years ago Closed 2 years ago

Consider using revocable events to schedule spell check jobs, and kill the old event when issuing a new one

Categories

(Core :: Spelling checker, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 684638

People

(Reporter: ehsan.akhgari, Unassigned)

References

Details

(Keywords: perf)

See bug 190147 comment 47.

This may give us some perf because we wouldn't get thousands of spell checker events in edgy test cases.  But a lot of care should go into this, as I'm not sure if the spell checker code can handle that (since it uses the ranges in mozSpellCheckingStatus to determine where to look for spelling errors, and coalescing spell checking jobs with non-adjacent ranges might be tricky.)

It seems we did something here, introducing mFullSpellCheckScheduled.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → DUPLICATE
See Also: → 856270, 684638
You need to log in before you can comment on or make changes to this bug.