Bug 425693 - Remove all isEmpty then return false portions from MSAA methods, patch=marco, r=me, a=beltzner

accessible/src/msaa/nsAccessibleWrap.cpp

@@ -323,9 +323,6 @@ __try {
     if (NS_FAILED(xpAccessible->GetValue(value)))
       return E_FAIL;
 
-    if (value.IsEmpty())
-      return S_FALSE;
-
     *pszValue = ::SysAllocStringLen(value.get(), value.Length());
     if (!*pszValue)
       return E_OUTOFMEMORY;
@@ -436,9 +433,6 @@ __try {
     description = NS_LITERAL_STRING("Description: ") + description;
   }
 
-  if (description.IsEmpty())
-    return S_FALSE;
-
   *pszDescription = ::SysAllocStringLen(description.get(),
                                         description.Length());
   return *pszDescription ? S_OK : E_OUTOFMEMORY;
@@ -581,9 +575,6 @@ __try {
     if (NS_FAILED(rv))
       return E_FAIL;
 
-    if (shortcut.IsEmpty())
-      return S_FALSE;
-
     *pszKeyboardShortcut = ::SysAllocStringLen(shortcut.get(),
                                                shortcut.Length());
     return *pszKeyboardShortcut ? S_OK : E_OUTOFMEMORY;
@@ -815,9 +806,6 @@ __try {
     if (NS_FAILED(xpAccessible->GetActionName(0, defaultAction)))
       return E_FAIL;
 
-    if (defaultAction.IsEmpty())
-      return S_FALSE;
-
     *pszDefaultAction = ::SysAllocStringLen(defaultAction.get(),
                                             defaultAction.Length());
     return *pszDefaultAction ? S_OK : E_OUTOFMEMORY;