Changeset 248952 in webkit

Timestamp:

Aug 21, 2019, 11:19:23 AM (7 years ago)

Author:

jer.noble@apple.com

Message:

[iOS] HLS streams disappear from Now Playing when paused on the lock screen
​https://bugs.webkit.org/show_bug.cgi?id=200951
<rdar://problem/54534301>

Reviewed by Eric Carlson.

The AVPlayerItemTracks will get recreated on occasion (during seeks, and when
changing buffering policy) which can result in a player which no longer reports
that it has audio, which in turn results in Now Playing no longer considering
the web page to be an eligable now playing application. Bridge this gap by also
taking AVPlayerItem.hasEnabledAudio into account when determining whether the
player has audio.

• platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:

(WebCore::MediaPlayerPrivateAVFoundationObjC::tracksChanged):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2019-08-21  Jer Noble  <jer.noble@apple.com>
+
+        [iOS] HLS streams disappear from Now Playing when paused on the lock screen
+        https://bugs.webkit.org/show_bug.cgi?id=200951
+        <rdar://problem/54534301>
+
+        Reviewed by Eric Carlson.
+
+        The AVPlayerItemTracks will get recreated on occasion (during seeks, and when
+        changing buffering policy) which can result in a player which no longer reports
+        that it has audio, which in turn results in Now Playing no longer considering
+        the web page to be an eligable now playing application. Bridge this gap by also
+        taking AVPlayerItem.hasEnabledAudio into account when determining whether the 
+        player has audio.
+
+        * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
+        (WebCore::MediaPlayerPrivateAVFoundationObjC::tracksChanged):
+
 2019-08-21  Adrian Perez de Castro  <aperez@igalia.com>
 

trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm

@@ -1837 +1837 @@
         hasVideo |= (m_visualGroup && m_visualGroup->selectedOption());
 
+        // HLS streams will occasionally recreate all their tracks; during seek and after
+        // buffering policy changes. "debounce" notifications which result in no enabled
+        // audio tracks by also taking AVPlayerItem.hasEnabledAudio into account when determining
+        // whethere there is any audio present.
+        hasAudio |= m_cachedHasEnabledAudio;
+
         // Always says we have video if the AVPlayerLayer is ready for diaplay to work around
         // an AVFoundation bug which causes it to sometimes claim a track is disabled even