Fixing bug 103689. Make sure that the right set of addresses are pre-populated in the compose window based on the items selected in the AdderssBook window. Also fixes the new card created from the AddressBook window's file new menu should reflect the right parent addressbook. r=ssu, sr=sspitzer

mailnews/addrbook/resources/content/abCommon.js

@@ -325,6 +325,9 @@ function AbNewMessage()
     var composeFields = Components.classes["@mozilla.org/messengercompose/composefields;1"].createInstance(Components.interfaces.nsIMsgCompFields);
     if (composeFields)
     {
+      if (DirPaneHasFocus())
+        composeFields.to = GetSelectedAddressesFromDirTree();
+      else
         composeFields.to = GetSelectedAddresses();
       params.composeFields = composeFields;
       msgComposeService.OpenComposeWindowWithParams(null, params);
@@ -367,26 +370,58 @@ function goToggleSplitter( id, elementID )
   }
 }
 
+// Generate a list of cards from the selected mailing list 
+// and get a comma separated list of card addresses. If the
+// item selected in the directory pane is not a mailing list,
+// an empty string is returned. 
+function GetSelectedAddressesFromDirTree() 
+{
+  var addresses = "";
+
+  var selectedItems = dirTree.selectedItems;
+  if (selectedItems.length == 1) {
+    var mailingListUri = selectedItems[0].getAttribute('id');
+    var directory = GetDirectoryFromURI(mailingListUri);
+    if (directory.isMailList) {
+      var listCardsCount = directory.addressLists.Count();
+      var cards = new Array(listCardsCount);
+      for ( var i = 0;  i < listCardsCount; i++ ) {
+        var card = directory.addressLists.GetElementAt(i);
+        card = card.QueryInterface(Components.interfaces.nsIAbCard);
+        cards[i] = card;
+      }
+      addresses = GetAddressesForCards(cards);
+    }
+  }
+  return addresses;
+}
+
 function GetSelectedAddresses()
 {
-  var selectedAddresses = "";
+  var selectedCards = GetSelectedAbCards();
+  return GetAddressesForCards(selectedCards);
+}
+
+// Generate a comma seperate list of addresses from a given
+// set of cards.
+function GetAddressesForCards(cards)
+{
+  var addresses = "";
 
-  var cards = GetSelectedAbCards();
   if (!cards)
-    return selectedAddresses;
+    return addresses;
 
   var count = cards.length;
   if (count > 0)
-    selectedAddresses += GenerateAddressFromCard(cards[0]);
+    addresses += GenerateAddressFromCard(cards[0]);
 
   for (var i = 1; i < count; i++) { 
     var generatedAddress = GenerateAddressFromCard(cards[i]);
 
     if (generatedAddress)
-      selectedAddresses += "," + generatedAddress;
+      addresses += "," + generatedAddress;
   }
-
+  return addresses;
-  return selectedAddresses;
 }
 
 function GetNumSelectedCards()
@@ -450,11 +485,9 @@ function GetSelectedAbCards()
       current++;
     }
   }
-
   return cards;
 }
 
-
 function SelectFirstAddressBook()
 {
   var children = dirTree.childNodes;
@@ -812,3 +845,9 @@ function GetParentDirectoryFromMailingListURI(abURI)
   return null;
 } 
 
+function DirPaneHasFocus()
+{
+  // returns true if diectory pane has the focus. Returns false, otherwise.
+  return (top.document.commandDispatcher.focusedElement == dirTree)
+}
+

mailnews/base/resources/content/mailABOverlay.xul

@@ -1,16 +1,21 @@
 <?xml version="1.0"?>
 
-<!-- retrieve generic commands -->
+<!DOCTYPE window SYSTEM "chrome://messenger/locale/mailOverlay.dtd">
-<?xul-overlay href="chrome://messenger/content/mailOverlay.xul"?>
 
 <overlay id="mailABOverlay"
          xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
 
-  <!-- addressbook specific UI items -->
+  <commandset id="tasksCommands">
+    <command id="cmd_newMessage" oncommand="AbNewMessage();"/>
+    <command id="cmd_newCard" oncommand="AbNewCard('dirTree');"/>
+  </commandset>
   <menupopup id="menu_NewPopup">
-    <menuitem id="menu_newMessage" position="5"/>
+    <menuitem id="menu_newMessage" position="5" label="&newMessageCmd.label;" key="key_newMessage" accesskey="&newMessageCmd.accesskey;" command="cmd_newMessage"/>
-    <menuitem id="menu_newCard" position="6"/>
+    <menuitem id="menu_newCard" position="6" label="&newCardCmd.label;" accesskey="&newCardCmd.accesskey;" command="cmd_newCard"/>
   </menupopup>
+  <keyset id="tasksKeys">
+    <key id="key_newMessage"   key="&newMessageCmd.key;" command="cmd_newMessage" modifiers="accel"/>
+  </keyset>
 
 </overlay>