Bug 1315298 - Remove redundant RefPtr = this in H264Converter.cpp. r?jya draft
authorJames Cheng <jacheng@mozilla.com>
Sat, 05 Nov 2016 00:14:13 +0800
changeset 433980 2e2bf171490f1ff4d81702e7652b4271ed37a8cd
parent 433942 9c6b3b3a119bc48f5d12e3161218e98adaa12627
child 535999 fec8adb8ecb228da51a936461ef747657d781cd6
push id34698
push userbmo:jacheng@mozilla.com
push dateFri, 04 Nov 2016 16:14:34 +0000
reviewersjya
bugs1315298
milestone52.0a1
Bug 1315298 - Remove redundant RefPtr = this in H264Converter.cpp. r?jya MozReview-Commit-ID: GzYes5HoCeM
dom/media/platforms/wrappers/H264Converter.cpp
--- a/dom/media/platforms/wrappers/H264Converter.cpp
+++ b/dom/media/platforms/wrappers/H264Converter.cpp
@@ -219,18 +219,16 @@ H264Converter::CreateDecoderAndInit(Medi
   UpdateConfigFromExtraData(extra_data);
 
   nsresult rv = CreateDecoder(/* DecoderDoctorDiagnostics* */ nullptr);
 
   if (NS_SUCCEEDED(rv)) {
     // Queue the incoming sample.
     mMediaRawSamples.AppendElement(aSample);
 
-    RefPtr<H264Converter> self = this;
-
     mInitPromiseRequest.Begin(mDecoder->Init()
       ->Then(AbstractThread::GetCurrent()->AsTaskQueue(), __func__, this,
              &H264Converter::OnDecoderInitDone,
              &H264Converter::OnDecoderInitFailed));
   }
   return rv;
 }