Bug 1495064 - part2 : don't request wakelock for video without audio track. r=jya

There are often websites using videos without an audio track as background image or as GIF-like image. For these cases,
we don't want to hold a wakelock.

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

--HG--
extra : moz-landing-system : lando

dom/html/HTMLMediaElement.cpp

@@ -7763,6 +7763,7 @@ HTMLMediaElement::SetMediaInfo(const MediaInfo& aInfo)
   if (mAudioChannelWrapper) {
     mAudioChannelWrapper->AudioCaptureStreamChangeIfNeeded();
   }
+  UpdateWakeLock();
 }
 
 void

dom/html/HTMLVideoElement.cpp

@@ -315,10 +315,19 @@ HTMLVideoElement::UpdateWakeLock()
   }
 }
 
+bool
+HTMLVideoElement::ShouldCreateVideoWakeLock() const
+{
+  // Make sure we only request wake lock for video with audio track, because
+  // video without audio track is often used as background image which seems no
+  // need to hold a wakelock.
+  return HasVideo() && HasAudio();
+}
+
 void
 HTMLVideoElement::CreateVideoWakeLockIfNeeded()
 {
-  if (!mScreenWakeLock && HasVideo()) {
+  if (!mScreenWakeLock && ShouldCreateVideoWakeLock()) {
     RefPtr<power::PowerManagerService> pmService =
       power::PowerManagerService::GetInstance();
     NS_ENSURE_TRUE_VOID(pmService);

dom/html/HTMLVideoElement.h

@@ -164,6 +164,7 @@ protected:
   void WakeLockRelease() override;
   void UpdateWakeLock() override;
 
+  bool ShouldCreateVideoWakeLock() const;
   void CreateVideoWakeLockIfNeeded();
   void ReleaseVideoWakeLockIfExists();