From 9abf52aa0fe4f9a0bd70d9544792c04e6d8869e3 Mon Sep 17 00:00:00 2001
From: John Lin <jolin@mozilla.com>
Date: Tue, 18 Dec 2018 19:46:16 +0000
Subject: [PATCH] Bug 1511607 - don't suspend background video decoding for
 unseekable media. r=jya

Differential Revision: https://phabricator.services.mozilla.com/D14821

--HG--
extra : moz-landing-system : lando
---
 dom/media/MediaDecoder.cpp | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/dom/media/MediaDecoder.cpp b/dom/media/MediaDecoder.cpp
index 97d84cf2f471..9cc5310f550e 100644
--- a/dom/media/MediaDecoder.cpp
+++ b/dom/media/MediaDecoder.cpp
@@ -974,6 +974,14 @@ void MediaDecoder::UpdateVideoDecodeMode() {
     return;
   }
 
+  // Seeking is required when leaving suspend mode.
+  if (!mMediaSeekable) {
+    LOG("UpdateVideoDecodeMode(), set Normal because the media is not "
+        "seekable");
+    mDecoderStateMachine->SetVideoDecodeMode(VideoDecodeMode::Normal);
+    return;
+  }
+
   // If mHasSuspendTaint is set, never suspend the video decoder.
   if (mHasSuspendTaint) {
     LOG("UpdateVideoDecodeMode(), set Normal because the element has been "