Title: [170485] trunk/Source/WebCore
Revision
170485
Author
[email protected]
Date
2014-06-26 11:29:57 -0700 (Thu, 26 Jun 2014)

Log 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.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (170484 => 170485)


--- trunk/Source/WebCore/ChangeLog	2014-06-26 18:17:36 UTC (rev 170484)
+++ trunk/Source/WebCore/ChangeLog	2014-06-26 18:29:57 UTC (rev 170485)
@@ -1,3 +1,15 @@
+2014-06-26  Eric Carlson  <[email protected]>
+
+        [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  <[email protected]>
 
         [MSE] Refactoring: Differentiate between SampleMap decode and presentation order methods

Modified: trunk/Source/WebCore/platform/audio/mac/MediaSessionManagerMac.cpp (170484 => 170485)


--- trunk/Source/WebCore/platform/audio/mac/MediaSessionManagerMac.cpp	2014-06-26 18:17:36 UTC (rev 170484)
+++ trunk/Source/WebCore/platform/audio/mac/MediaSessionManagerMac.cpp	2014-06-26 18:29:57 UTC (rev 170485)
@@ -55,7 +55,8 @@
 #endif
 
 #if PLATFORM(IOS)
-    AudioSession::sharedSession().setActive(hasActive());
+    if (hasActive())
+        AudioSession::sharedSession().setActive(true);
 
     if (!Settings::shouldManageAudioSessionCategory())
         return;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to