Title: [276824] trunk/Source/WebKit
Revision
276824
Author
[email protected]
Date
2021-04-29 17:20:32 -0700 (Thu, 29 Apr 2021)

Log Message

AVAudioSession throws an exception when WebKit tries to set its routeSharingPolicy to "independent"
https://bugs.webkit.org/show_bug.cgi?id=225213

Reviewed by Jer Noble.

AVAudioSession does not expect WebKit to set its routeSharingPolicy
to AVAudioSessionRouteSharingPolicyIndependent(RouteSharingPolicy::Independent).

* GPUProcess/media/RemoteAudioSessionProxyManager.cpp:
(WebKit::RemoteAudioSessionProxyManager::updateCategory):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (276823 => 276824)


--- trunk/Source/WebKit/ChangeLog	2021-04-30 00:12:00 UTC (rev 276823)
+++ trunk/Source/WebKit/ChangeLog	2021-04-30 00:20:32 UTC (rev 276824)
@@ -1,3 +1,16 @@
+2021-04-29  Peng Liu  <[email protected]>
+
+        AVAudioSession throws an exception when WebKit tries to set its routeSharingPolicy to "independent"
+        https://bugs.webkit.org/show_bug.cgi?id=225213
+
+        Reviewed by Jer Noble.
+
+        AVAudioSession does not expect WebKit to set its routeSharingPolicy
+        to AVAudioSessionRouteSharingPolicyIndependent(RouteSharingPolicy::Independent).
+
+        * GPUProcess/media/RemoteAudioSessionProxyManager.cpp:
+        (WebKit::RemoteAudioSessionProxyManager::updateCategory):
+
 2021-04-29  Brent Fulgham  <[email protected]>
 
         [iOS] Remove unused logd-diagnostic-client rules

Modified: trunk/Source/WebKit/GPUProcess/media/RemoteAudioSessionProxyManager.cpp (276823 => 276824)


--- trunk/Source/WebKit/GPUProcess/media/RemoteAudioSessionProxyManager.cpp	2021-04-30 00:12:00 UTC (rev 276823)
+++ trunk/Source/WebKit/GPUProcess/media/RemoteAudioSessionProxyManager.cpp	2021-04-30 00:20:32 UTC (rev 276824)
@@ -95,14 +95,13 @@
     else
         category = AudioSession::None;
 
+    policy = RouteSharingPolicy::Default;
     if (policyCounts.contains(RouteSharingPolicy::LongFormVideo))
         policy = RouteSharingPolicy::LongFormVideo;
     else if (policyCounts.contains(RouteSharingPolicy::LongFormAudio))
         policy = RouteSharingPolicy::LongFormAudio;
     else if (policyCounts.contains(RouteSharingPolicy::Independent))
-        policy = RouteSharingPolicy::Independent;
-    else
-        policy = RouteSharingPolicy::Default;
+        ASSERT_NOT_REACHED();
 
     m_session->setCategory(category, policy);
 }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to