Changeset 170485 in webkit

Timestamp:

Jun 26, 2014 11:29:57 AM (10 years ago)

Author:

eric.carlson@apple.com

Message:

[iOS] no need to deactivate audio session
​https://bugs.webkit.org/show_bug.cgi?id=134350

Reviewed by Jer Noble.

• platform/audio/mac/MediaSessionManagerMac.cpp:

(MediaSessionManager::updateSessionState): Don't bother calling

AudioSession::setActive(false), WebCore's audio session will be deactivated
automatically if another application begins playing audio.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/audio/mac/MediaSessionManagerMac.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2014-06-26  Eric Carlson  <eric.carlson@apple.com>
+
+        [iOS] no need to deactivate audio session
+        https://bugs.webkit.org/show_bug.cgi?id=134350
+
+        Reviewed by Jer Noble.
+
+        * platform/audio/mac/MediaSessionManagerMac.cpp:
+        (MediaSessionManager::updateSessionState): Don't bother calling 
+            AudioSession::setActive(false), WebCore's audio session will be deactivated 
+            automatically if another application begins playing audio.
+
 2014-06-26  Jer Noble  <jer.noble@apple.com>
 

trunk/Source/WebCore/platform/audio/mac/MediaSessionManagerMac.cpp

@@ -56 +56 @@
 
 #if PLATFORM(IOS)
-    AudioSession::sharedSession().setActive(hasActive());
+    if (hasActive())
+        AudioSession::sharedSession().setActive(true);
 
     if (!Settings::shouldManageAudioSessionCategory())