Bug 814386 - ASSERTION: No parent accessible. Should we really assert here, r=tbsaunde
authorAlexander Surkov <surkov.alexander@gmail.com>
Thu, 29 Nov 2012 14:14:04 +0900
changeset 123562 5948fb7390ef95c3c1b44a41a7214351558ef338
parent 123561 676bf833f816e24d7c24650f13c89254e3c9f698
child 123563 7a41fb508d8923279e528c9b7f2dc88f362cb6b2
push id2151
push userlsblakk@mozilla.com
push dateTue, 19 Feb 2013 18:06:57 +0000
treeherdermozilla-beta@4952e88741ec [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstbsaunde
bugs814386
milestone20.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 814386 - ASSERTION: No parent accessible. Should we really assert here, r=tbsaunde
accessible/src/msaa/AccessibleWrap.cpp
--- a/accessible/src/msaa/AccessibleWrap.cpp
+++ b/accessible/src/msaa/AccessibleWrap.cpp
@@ -182,23 +182,18 @@ AccessibleWrap::get_accParent( IDispatch
                                                          IID_IAccessible,
                                                          (void**)ppdispParent))) {
         return S_OK;
       }
     }
   }
 
   Accessible* xpParentAcc = Parent();
-  if (!xpParentAcc) {
-    if (IsApplication())
-      return S_OK;
-
-    NS_ERROR("No parent accessible. Should we really assert here?");
-    return E_UNEXPECTED;
-  }
+  if (!xpParentAcc)
+    return S_FALSE;
 
   *ppdispParent = NativeAccessible(xpParentAcc);
   return S_OK;
 
   A11Y_TRYBLOCK_END
 }
 
 STDMETHODIMP