Don't flush pending notifications on the spellchecker

RESOLVED FIXED in mozilla1.8.1beta2

Status

()

P1
normal
RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: brettw, Assigned: brettw)

Tracking

({fixed1.8.1})

1.8 Branch
mozilla1.8.1beta2
fixed1.8.1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

13 years ago
This call was left over from when the spellchecker dealt with frames instead of nodes. Now it just slows things down.
(Assignee)

Comment 1

13 years ago
Created attachment 229668 [details] [diff] [review]
Patch
Attachment #229668 - Flags: review?(roc)
Attachment #229668 - Flags: approval1.8.1?
(Assignee)

Updated

13 years ago
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]
(Assignee)

Comment 4

13 years ago
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+
(Assignee)

Updated

13 years ago
Status: NEW → RESOLVED
Last Resolved: 13 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.