bug 241061 Action name exported by mozilla is messed up in at-poke

r = pkw@us.ibm.com
sr= henry.jia@sun.com

accessible/src/atk/nsMaiInterfaceAction.cpp

@@ -139,16 +139,11 @@ getNameCB(AtkAction *aAction, gint aActionIndex)
                        accWrap->GetMaiInterface(MAI_INTERFACE_ACTION));
     NS_ENSURE_TRUE(action, nsnull);
 
-    const char *name = action->GetName();
-    if (!name || !*name) {
-        nsAutoString autoStr;
-        nsresult rv = accWrap->GetActionName(aActionIndex, autoStr);
-        NS_ENSURE_SUCCESS(rv, nsnull);
-
-        action->SetName(autoStr);
-        name = action->GetName();
-    }
-    return name;
+    nsAutoString autoStr;
+    nsresult rv = accWrap->GetActionName(aActionIndex, autoStr);
+    NS_ENSURE_SUCCESS(rv, nsnull);
+    action->SetName(autoStr);
+    return action->GetName();
 }
 
 const gchar *

accessible/src/atk/nsMaiInterfaceAction.h

@@ -54,16 +54,16 @@ public:
     virtual const GInterfaceInfo *GetInterfaceInfo();
 
     const char *GetName() {
-        return NS_ConvertUCS2toUTF8(mName).get(); 
+        return mName.get();
     }
-    void SetName(nsAString &aString) { mName = aString; }
+    void SetName(nsAString &aString) { mName = NS_ConvertUCS2toUTF8(aString); }
 
     const char *GetKeyBinding() {
         return NS_ConvertUCS2toUTF8(mKeyBinding).get(); 
     }
     void SetKeyBinding(nsAString &aString) { mKeyBinding = aString; }
 private:
-    nsString mName;
+    nsCString mName;
     nsString mKeyBinding;
 };