Bug 1424191 - set mImages[i] to nullptr in MediaEngineRemoteVideoSource::Start(). r?pehrsons draft
authorMunro Mengjue Chiang <mchiang@mozilla.com>
Tue, 12 Dec 2017 14:56:52 -0600
changeset 712053 ff21dad3539a69eec84a13aec99af172fd6f2030
parent 710980 93b37aa497c48a6e28a9463eeb753b2ce3964f42
child 743965 ca4bf4882a4de84c2844e6647f4661248a0ec1a6
push id93240
push userbmo:mchiang@mozilla.com
push dateFri, 15 Dec 2017 16:16:09 +0000
reviewerspehrsons
bugs1424191
milestone59.0a1
Bug 1424191 - set mImages[i] to nullptr in MediaEngineRemoteVideoSource::Start(). r?pehrsons MozReview-Commit-ID: LK3gFynesoE
dom/media/webrtc/MediaEngineRemoteVideoSource.cpp
--- a/dom/media/webrtc/MediaEngineRemoteVideoSource.cpp
+++ b/dom/media/webrtc/MediaEngineRemoteVideoSource.cpp
@@ -185,17 +185,17 @@ MediaEngineRemoteVideoSource::Start(Sour
   }
 
   {
     MonitorAutoLock lock(mMonitor);
     mSources.AppendElement(aStream);
     mPrincipalHandles.AppendElement(aPrincipalHandle);
     mTargetCapabilities.AppendElement(mTargetCapability);
     mHandleIds.AppendElement(mHandleId);
-    mImages.AppendElement(mImageContainer->CreatePlanarYCbCrImage());
+    mImages.AppendElement(nullptr);
 
     MOZ_ASSERT(mSources.Length() == mPrincipalHandles.Length());
     MOZ_ASSERT(mSources.Length() == mTargetCapabilities.Length());
     MOZ_ASSERT(mSources.Length() == mHandleIds.Length());
     MOZ_ASSERT(mSources.Length() == mImages.Length());
   }
 
   aStream->AddTrack(aID, 0, new VideoSegment(), SourceMediaStream::ADDTRACK_QUEUED);