132108 - Remove temporary patch for bug 26528

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect)

Description

[Jean-Francois Ducarroz]

Now that bug 26528 has been finally fixed, we can remove the API
nsIMsgCompose::ResetNodeEventHandlers. That will slightly speed up new message
compose.

Comment 1

[Jean-Francois Ducarroz]

nominating nsbeta1. This is an easy way to improve the messagecompose window by
removing superfluous code.

Comment 2

[Michael Buckland]

Discussed at Mail News bug meeting with Engineering, QA Mktng and PjM.  Decided
to minus this bug.

Comment 3

[selmer (gone)]

Please update this bug with an [adt1] - [adt3] impact rating (or take it off the
list if it doesn't even rate adt3.)  Thanks!

Comment 4

[(not reading, please use [e-mail] instead)]

taking all of varada's bugs.

Comment 5

[[e-mail]]

Attachment: Proposed patch

Comment 6

[Jean-Francois Ducarroz]

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

Comment 7

[(not reading, please use [e-mail] instead)]

Comment on attachment 113387 [details] [diff] [review]
Proposed patch

sr=sspitzer, assuming you tested.  I think this can wait until after 1.3 beta.

Comment 8

[[e-mail]]

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

Comment 9

[[e-mail]]

Fix checked in.

Comment 10

[[e-mail] (gone - use [e-mail] instead)]

Verified.

Patches have been in CVS since Feb 24th.