98657 - remove <commandset id="composerSaveMenuItems"/> from messengercompose.xul

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect)

Description

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

remove <commandset id="composerSaveMenuItems"/> from messengercompose.xul

having it causes us to overlay a some editor command stuff that we don't care
about.  (see
http://lxr.mozilla.org/mozilla/source/editor/ui/composer/content/editorOverlay.xul#122)

specificially, we do some command updating on the create and save events.

we've got File | Save (and a save button), but I'm pretty sure none of it is
hooked up to how editor does save, we do save all on our own.

obviously, before you remove it should should confirm that I'm right that we
don't need it. 
this won't be a huge performance win, but we should still get to it at some point.

Comment 1

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

taking, I've got this in my tree.  not a big win, but still something.

Comment 2

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

fixed

Comment 3

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

Attachment: Comparison of 11-2 and 10-26 builds (win32)

Comment 4

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

Ignore the typo (10-16 is actually 10-26).

Verified FIXED, based on the logs