--- a/accessible/src/msaa/nsDocAccessibleWrap.cpp
+++ b/accessible/src/msaa/nsDocAccessibleWrap.cpp
@@ -199,17 +199,17 @@ NS_IMETHODIMP nsDocAccessibleWrap::FireA
 }
 
 STDMETHODIMP nsDocAccessibleWrap::get_URL(/* [out] */ BSTR __RPC_FAR *aURL)
 {
 __try {
   *aURL = NULL;
 
   nsAutoString URL;
-  nsresult rv = GetURL(URL));
+  nsresult rv = GetURL(URL);
   if (NS_FAILED(rv))
     return E_FAIL;
 
   if (URL.IsEmpty())
     return S_FALSE;
 
   *aURL = ::SysAllocStringLen(URL.get(), URL.Length());
   return *aURL ? S_OK : E_OUTOFMEMORY;
@@ -219,17 +219,17 @@ STDMETHODIMP nsDocAccessibleWrap::get_UR
 }
 
 STDMETHODIMP nsDocAccessibleWrap::get_title( /* [out] */ BSTR __RPC_FAR *aTitle)
 {
 __try {
   *aTitle = NULL;
 
   nsAutoString title;
-  nsresult rv = GetTitle(title));
+  nsresult rv = GetTitle(title);
   if (NS_FAILED(rv))
     return E_FAIL;
 
   if (title.IsEmpty())
     return S_FALSE;
 
   *aTitle = ::SysAllocStringLen(title.get(), title.Length());
   return *aTitle ? S_OK : E_OUTOFMEMORY;
@@ -239,17 +239,17 @@ STDMETHODIMP nsDocAccessibleWrap::get_ti
 }
 
 STDMETHODIMP nsDocAccessibleWrap::get_mimeType(/* [out] */ BSTR __RPC_FAR *aMimeType)
 {
 __try {
   *aMimeType = NULL;
 
   nsAutoString mimeType;
-  nsresult rv = GetMimeType(mimeType));
+  nsresult rv = GetMimeType(mimeType);
   if (NS_FAILED(rv))
     return E_FAIL;
 
   if (mimeType.IsEmpty())
     return S_FALSE;
 
   *aMimeType = ::SysAllocStringLen(mimeType.get(), mimeType.Length());
   return *aMimeType ? S_OK : E_OUTOFMEMORY;
@@ -259,17 +259,17 @@ STDMETHODIMP nsDocAccessibleWrap::get_mi
 }
 
 STDMETHODIMP nsDocAccessibleWrap::get_docType(/* [out] */ BSTR __RPC_FAR *aDocType)
 {
 __try {
   *aDocType = NULL;
 
   nsAutoString docType;
-  nsresult rv = GetDocType(docType));
+  nsresult rv = GetDocType(docType);
   if (NS_FAILED(rv))
     return E_FAIL;
 
   if (docType.IsEmpty())
     return S_FALSE;
 
   *aDocType = ::SysAllocStringLen(docType.get(), docType.Length());
   return *aDocType ? S_OK : E_OUTOFMEMORY;
@@ -283,17 +283,17 @@ STDMETHODIMP nsDocAccessibleWrap::get_na
 {
 __try {
   *aNameSpaceURI = NULL;
 
   if (aNameSpaceID < 0)
     return E_INVALIDARG;  // -1 is kNameSpaceID_Unknown
 
   nsAutoString nameSpaceURI;
-  nsresult rv = GetNameSpaceURIForID(aNameSpaceID, nameSpaceURI));
+  nsresult rv = GetNameSpaceURIForID(aNameSpaceID, nameSpaceURI);
   if (NS_FAILED(rv))
     return E_FAIL;
 
   if (nameSpaceURI.IsEmpty())
     return S_FALSE;
 
   *aNameSpaceURI = ::SysAllocStringLen(nameSpaceURI.get(),
                                        nameSpaceURI.Length());