Bug 1321497 - correct the logic of resuming from AudioChannel; r=alwu,jwwang
authorKaku Kuo <kaku@mozilla.com>
Thu, 01 Dec 2016 16:30:18 +0800
changeset 325318 34a2a7ffa7b3bf3992062526e99b3594391277cf
parent 325317 181184c5f80eb272aa8cdbe8fca8a909c3103b99
child 325319 61d6fe7c63903f8f3d5ba8e9af58f6893f763167
push id31051
push usercbook@mozilla.com
push dateThu, 08 Dec 2016 15:15:04 +0000
treeherdermozilla-central@64e024fe4471 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersalwu, jwwang
bugs1321497
milestone53.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1321497 - correct the logic of resuming from AudioChannel; r=alwu,jwwang MozReview-Commit-ID: JxeqdzADlrU
dom/html/HTMLMediaElement.cpp
--- a/dom/html/HTMLMediaElement.cpp
+++ b/dom/html/HTMLMediaElement.cpp
@@ -787,17 +787,17 @@ private:
 
     NotifyAudioPlaybackChanged(
       AudioChannelService::AudibleChangedReasons::ePauseStateChanged);
   }
 
   void
   Resume()
   {
-    if (!IsSuspended() && mOwner->Paused()) {
+    if (!IsSuspended()) {
       MOZ_LOG(AudioChannelService::GetAudioChannelLog(), LogLevel::Debug,
              ("HTMLMediaElement::AudioChannelAgentCallback, ResumeFromAudioChannel, "
               "this = %p, Error : resume without suspended!\n", this));
       return;
     }
 
     SetSuspended(nsISuspendedTypes::NONE_SUSPENDED);
     nsresult rv = mOwner->Play();