From 9709651f993e3c0e601ba22a25503cc9b437ffb3 Mon Sep 17 00:00:00 2001 From: Nathan Froyd Date: Tue, 20 Sep 2016 21:29:28 -0400 Subject: [PATCH] Bug 1303910 - remove superfluous FallibleTArray in Blob.cpp; r=baku This array only exists to be initialized fallibly so we can swap it into the infallible array we actually want to use. But we can fallibly initialize the infallible array directly, so let's do that. --- dom/ipc/Blob.cpp | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/dom/ipc/Blob.cpp b/dom/ipc/Blob.cpp index 2292c409e428..e29d467e7c28 100644 --- a/dom/ipc/Blob.cpp +++ b/dom/ipc/Blob.cpp @@ -823,14 +823,11 @@ CreateBlobImpl(const nsTArray& aBlobDatas, return blobImpl.forget(); } - FallibleTArray> fallibleBlobImpls; - if (NS_WARN_IF(!fallibleBlobImpls.SetLength(aBlobDatas.Length(), fallible))) { + nsTArray> blobImpls; + if (NS_WARN_IF(!blobImpls.SetLength(aBlobDatas.Length(), fallible))) { return nullptr; } - nsTArray> blobImpls; - fallibleBlobImpls.SwapElements(blobImpls); - const bool hasRecursed = aMetadata.mHasRecursed; aMetadata.mHasRecursed = true;