Bug #257378 --> L10n teams should be able to translate reply header "on date" through *.properties file instead of using a hard coded string in mailnews.js

sr=bienvenu

mailnews/compose/resources/locale/en-US/composeMsgs.properties

@@ -301,6 +301,11 @@ sendMessageCheckSendButtonLabel=Send
 ## <author> wrote:
 mailnews.reply_header_authorwrote=%s wrote
 
+## reply header in composeMsg
+## <author> wrote:
+mailnews.reply_header_authorwrote=%s wrote
+mailnews.reply_header_ondate=On %s
+
 ## reply header in composeMsg
 ## user specified 
 mailnews.reply_header_originalmessage=-------- Original Message --------

mailnews/compose/src/nsMsgCompose.cpp

@@ -147,7 +147,7 @@ static nsresult GetReplyHeaderInfo(PRInt32* reply_header_type,
     if (NS_FAILED(rv) || !*reply_header_authorwrote)
       *reply_header_authorwrote = nsCRT::strdup(NS_LITERAL_STRING("%s wrote").get());
 
-    rv = prefs->CopyUnicharPref("mailnews.reply_header_ondate", reply_header_ondate);
+    rv = prefs->GetLocalizedUnicharPref("mailnews.reply_header_ondate", reply_header_ondate);
     if (NS_FAILED(rv) || !*reply_header_ondate)
       *reply_header_ondate = nsCRT::strdup(NS_LITERAL_STRING("On %s").get());
 

mailnews/mailnews.js

@@ -187,7 +187,8 @@ pref("mailnews.reply_header_type",          1);
 // locale which affects date format, set empty string to use application default locale
 pref("mailnews.reply_header_locale",        "");
 pref("mailnews.reply_header_authorwrote",   "chrome://messenger/locale/messengercompose/composeMsgs.properties");
-pref("mailnews.reply_header_ondate",        "On %s");
+pref("mailnews.reply_header_ondate",        "chrome://messenger/locale/messengercompose/composeMsgs.properties");
+
 // separator to separate between date and author
 pref("mailnews.reply_header_separator",     ", ");
 pref("mailnews.reply_header_colon",         ":");