bug 1332690 - work around non existant proxies
authorTrevor Saunders <tbsaunde@tbsaunde.org>
Tue, 21 Feb 2017 11:18:32 -0500
changeset 373112 4e41289d88c7cf5fd539083ead2bf415ecd7a249
parent 373111 e3699678d46e5393c96b49ca55641e67075ab1cc
child 373113 775d6573482d7d0af7431ae3f54d1b9662fefb26
push id10863
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 23:02:23 +0000
treeherdermozilla-aurora@0931190cd725 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1332690
milestone54.0a1
bug 1332690 - work around non existant proxies
accessible/ipc/DocAccessibleParent.cpp
--- a/accessible/ipc/DocAccessibleParent.cpp
+++ b/accessible/ipc/DocAccessibleParent.cpp
@@ -406,18 +406,23 @@ DocAccessibleParent::RecvBindChildDoc(PD
 
 ipc::IPCResult
 DocAccessibleParent::AddChildDoc(DocAccessibleParent* aChildDoc,
                                  uint64_t aParentID, bool aCreating)
 {
   // We do not use GetAccessible here because we want to be sure to not get the
   // document it self.
   ProxyEntry* e = mAccessibles.GetEntry(aParentID);
-  if (!e)
+  if (!e) {
+#ifdef DEBUG
     return IPC_FAIL(this, "binding to nonexistant proxy!");
+#else
+    return IPC_OK();
+#endif
+  }
 
   ProxyAccessible* outerDoc = e->mProxy;
   MOZ_ASSERT(outerDoc);
 
   // OuterDocAccessibles are expected to only have a document as a child.
   // However for compatibility we tolerate replacing one document with another
   // here.
   if (outerDoc->ChildrenCount() > 1 ||