Closed
Bug 132108
Opened 22 years ago
Closed 22 years ago
Remove temporary patch for bug 26528
Categories
(MailNews Core :: Composition, defect)
MailNews Core
Composition
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.4alpha
People
(Reporter: bugzilla, Assigned: sspitzer)
Details
(Keywords: perf)
Attachments
(1 file)
6.07 KB,
patch
|
bugzilla
:
review+
sspitzer
:
superreview+
|
Details | Diff | Splinter Review |
Now that bug 26528 has been finally fixed, we can remove the API nsIMsgCompose::ResetNodeEventHandlers. That will slightly speed up new message compose.
Reporter | ||
Comment 1•22 years ago
|
||
nominating nsbeta1. This is an easy way to improve the messagecompose window by removing superfluous code.
OS: Windows 2000 → All
QA Contact: sheelar → stephend
Hardware: PC → All
Comment 2•22 years ago
|
||
Discussed at Mail News bug meeting with Engineering, QA Mktng and PjM. Decided to minus this bug.
Comment 3•22 years ago
|
||
Please update this bug with an [adt1] - [adt3] impact rating (or take it off the list if it doesn't even rate adt3.) Thanks!
Updated•22 years ago
|
Keywords: mozilla1.1
Comment 5•22 years ago
|
||
Updated•22 years ago
|
Attachment #113387 -
Flags: superreview?(sspitzer)
Attachment #113387 -
Flags: review?(ducarroz)
Reporter | ||
Comment 6•22 years ago
|
||
Comment on attachment 113387 [details] [diff] [review] Proposed patch I presume you have carefully tested the addressing widget as well the adressbook list. R=ducarroz
Attachment #113387 -
Flags: review?(ducarroz) → review+
Assignee | ||
Comment 7•22 years ago
|
||
Comment on attachment 113387 [details] [diff] [review] Proposed patch sr=sspitzer, assuming you tested. I think this can wait until after 1.3 beta.
Attachment #113387 -
Flags: superreview?(sspitzer) → superreview+
Comment 8•22 years ago
|
||
Actually the address list code never called (commented out). But I've tested the addressing widget on Linux and Windows 98.
Updated•22 years ago
|
Target Milestone: mozilla1.0 → mozilla1.4alpha
Comment 9•22 years ago
|
||
Fix checked in.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Verified. Patches have been in CVS since Feb 24th.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•