Bug 952840, part 2 - Get rid of one goto in XPCNativeSet::GetNewOrUsed. r=bholley
arr.get() only returns null if the second argument to its constructor is null,
but here it is being infallibly allocated.
--- a/js/xpconnect/src/XPCWrappedNativeInfo.cpp
+++ b/js/xpconnect/src/XPCWrappedNativeInfo.cpp
@@ -474,18 +474,16 @@ XPCNativeSet::GetNewOrUsed(nsIClassInfo*
MOZ_ASSERT((iidCount && iidArray) || !(iidCount || iidArray), "GetInterfaces returned bad array");
// !!! from here on we only exit through the 'out' label !!!
if (iidCount) {
AutoMarkingNativeInterfacePtrArrayPtr
arr(cx, new XPCNativeInterface*[iidCount], iidCount, true);
- if (!arr)
- goto out;
interfaceArray = arr;
XPCNativeInterface** currentInterface = interfaceArray;
nsIID** currentIID = iidArray;
uint16_t interfaceCount = 0;
for (uint32_t i = 0; i < iidCount; i++) {