Bug 1723391 - Use RefPtr in ContentChild::RecvGetFilesResponse. r=smaug, a=RyanVM

This commit is contained in:
roytam1 2021-09-15 11:36:54 +08:00
commit 8d7fa9bdd6

View file

@ -2817,10 +2817,10 @@ bool
ContentChild::RecvGetFilesResponse(const nsID& aUUID,
const GetFilesResponseResult& aResult)
{
GetFilesHelperChild* child = mGetFilesPendingRequests.GetWeak(aUUID);
RefPtr<GetFilesHelperChild> child;
// This object can already been deleted in case DeleteGetFilesRequest has
// been called when the response was sending by the parent.
if (!child) {
if (!mGetFilesPendingRequests.Remove(aUUID, getter_AddRefs(child))) {
return true;
}
@ -2841,7 +2841,6 @@ ContentChild::RecvGetFilesResponse(const nsID& aUUID,
child->Finished(succeeded ? NS_OK : NS_ERROR_OUT_OF_MEMORY);
}
mGetFilesPendingRequests.Remove(aUUID);
return true;
}