Bug 1454630 - P4. Only wait for CDMProxy if actually needed for the given decoder. r?bryce draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Mon, 28 May 2018 15:10:19 +0200
changeset 800556 863dc369b417e825eac8ffd547a9bb9fde76469a
parent 800555 4cf3d9f1f1a4e3213db50b0e8d6454d4783d0141
push id111396
push userbmo:jyavenard@mozilla.com
push dateMon, 28 May 2018 13:31:40 +0000
reviewersbryce
bugs1454630
milestone62.0a1
Bug 1454630 - P4. Only wait for CDMProxy if actually needed for the given decoder. r?bryce If the particular track isn't encrypted, there's no need to wait for a CDMProxy. MozReview-Commit-ID: DPbvbwsO58N
dom/media/MediaFormatReader.cpp
--- a/dom/media/MediaFormatReader.cpp
+++ b/dom/media/MediaFormatReader.cpp
@@ -1496,17 +1496,18 @@ MediaFormatReader::PrepareToSetCDMForTra
   }
   ScheduleUpdate(aTrack);
 }
 
 bool
 MediaFormatReader::IsDecoderWaitingForCDM(TrackType aTrack)
 {
   MOZ_ASSERT(OnTaskQueue());
-  return IsEncrypted() && mSetCDMForTracks.contains(aTrack) && !mCDMProxy;
+  return GetDecoderData(aTrack).IsEncrypted() &&
+         mSetCDMForTracks.contains(aTrack) && !mCDMProxy;
 }
 
 RefPtr<SetCDMPromise>
 MediaFormatReader::SetCDMProxy(CDMProxy* aProxy)
 {
   MOZ_ASSERT(OnTaskQueue());
   LOGV("SetCDMProxy (%p)", aProxy);