Closed Bug 262832 Opened 20 years ago Closed 20 years ago

nsTextServicesDocument has slow copy/pasted code

Categories

(Core :: DOM: Editor, defect)

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

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.8alpha5

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

(Keywords: perf)

nsTextServicesDocument has a copy/paste of the ComparePoints function that lives in nsRange.cpp. The thing is, this function is really slow (see bug 262764). Why can't this just use nsIRangeUtils so that improvements to the core code will automatically be benefited from?
Whiteboard: [good first bug]
This probably depends on bug 262833 -- once that's fixed we could just use the nsRangeUtils service here without even needing a local ptr.
Depends on: 262833
Assignee: mozeditor → bzbarsky
Whiteboard: [good first bug]
Fixed on trunk.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.8alpha5
You need to log in before you can comment on or make changes to this bug.