Closed Bug 345054 Opened 15 years ago Closed 15 years ago

Don't flush pending notifications on the spellchecker

Categories

(Core :: Spelling checker, defect, P1)

1.8 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla1.8.1beta2

People

(Reporter: brettw, Assigned: brettw)

References

Details

(Keywords: fixed1.8.1)

Attachments

(1 file)

This call was left over from when the spellchecker dealt with frames instead of nodes. Now it just slows things down.
Attached patch PatchSplinter Review
Attachment #229668 - Flags: review?(roc)
Attachment #229668 - Flags: approval1.8.1?
Priority: -- → P1
Target Milestone: --- → mozilla1.8.1beta2
Apparently roc's laptop has died, you you might want to lend him one, or invite him over to your office for an over-the-shoulder review :)
Can we get this landed on trunk for some bake-time?
Whiteboard: [checkin needed]
Checked in on trunk, leaving open so we don't forget about branch.
Whiteboard: [checkin needed] → baking on trunk before branch checkin
Comment on attachment 229668 [details] [diff] [review]
Patch

a=drivers, please land on the branch
Attachment #229668 - Flags: approval1.8.1? → approval1.8.1+
Status: NEW → RESOLVED
Closed: 15 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Whiteboard: baking on trunk before branch checkin
You need to log in before you can comment on or make changes to this bug.