Remove temporary patch for bug 26528

VERIFIED FIXED in mozilla1.4alpha

Status

MailNews Core
Composition
VERIFIED FIXED
16 years ago
9 years ago

People

(Reporter: Jean-Francois Ducarroz, Assigned: (not reading, please use seth@sspitzer.org instead))

Tracking

({perf})

Trunk
mozilla1.4alpha

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

6.07 KB, patch
Jean-Francois Ducarroz
: review+
(not reading, please use seth@sspitzer.org instead)
: superreview+
Details | Diff | Splinter Review
(Reporter)

Description

16 years ago
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

16 years ago
nominating nsbeta1. This is an easy way to improve the messagecompose window by
removing superfluous code.
Keywords: nsbeta1, perf
Target Milestone: --- → mozilla1.0
OS: Windows 2000 → All
QA Contact: sheelar → stephend
Hardware: PC → All

Comment 2

16 years ago
Discussed at Mail News bug meeting with Engineering, QA Mktng and PjM.  Decided
to minus this bug.
Keywords: nsbeta1 → nsbeta1-

Comment 3

16 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

16 years ago
Keywords: mozilla1.1
taking all of varada's bugs.
Assignee: varada → sspitzer

Comment 5

15 years ago
Created attachment 113387 [details] [diff] [review]
Proposed patch

Updated

15 years ago
Attachment #113387 - Flags: superreview?(sspitzer)
Attachment #113387 - Flags: review?(ducarroz)
(Reporter)

Comment 6

15 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+
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

15 years ago
Actually the address list code never called (commented out).
But I've tested the addressing widget on Linux and Windows 98.

Updated

15 years ago
Target Milestone: mozilla1.0 → mozilla1.4alpha

Comment 9

15 years ago
Fix checked in.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
Verified.

Patches have been in CVS since Feb 24th.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.