Bug 1688490: Make GetNativeFromGeckoAccessible null-safe. r=eeejay, a=pascalc

Differential Revision: https://phabricator.services.mozilla.com/D103813

Author: MReschenberg

accessible/mac/mozAccessible.h

@@ -27,7 +27,9 @@ namespace a11y {
 
 inline mozAccessible* GetNativeFromGeckoAccessible(
     mozilla::a11y::AccessibleOrProxy aAccOrProxy) {
-  MOZ_ASSERT(!aAccOrProxy.IsNull(), "Cannot get native from null accessible");
+  if (aAccOrProxy.IsNull()) {
+    return nil;
+  }
   if (Accessible* acc = aAccOrProxy.AsAccessible()) {
     mozAccessible* native = nil;
     acc->GetNativeInterface((void**)&native);

accessible/mac/mozSelectableElements.mm

@@ -263,10 +263,12 @@ - (BOOL)moxIgnoreWithParent:(mozAccessible*)parent {
   // we override this method.
   AccessibleOrProxy parentAcc = [parent geckoAccessible];
   if (!parentAcc.IsNull()) {
-    mozAccessible* directGrandparent =
-        GetNativeFromGeckoAccessible(parentAcc.Parent());
-    if ([directGrandparent isKindOfClass:[MOXWebAreaAccessible class]]) {
-      return [parent moxIgnoreWithParent:directGrandparent];
+    AccessibleOrProxy grandparentAcc = parentAcc.Parent();
+    if (mozAccessible* directGrandparent =
+            GetNativeFromGeckoAccessible(grandparentAcc)) {
+      if ([directGrandparent isKindOfClass:[MOXWebAreaAccessible class]]) {
+        return [parent moxIgnoreWithParent:directGrandparent];
+      }
     }
   }