Title: [228426] tags/Safari-605.1.27.1/Source/WebCore
Revision
228426
Author
jmarc...@apple.com
Date
2018-02-13 11:08:51 -0800 (Tue, 13 Feb 2018)

Log Message

Apply patch. rdar://problem/37485750

    CoreAudioCaptureSourceIOSListener should be a WebProcess singleton

Modified Paths

Diff

Modified: tags/Safari-605.1.27.1/Source/WebCore/ChangeLog (228425 => 228426)


--- tags/Safari-605.1.27.1/Source/WebCore/ChangeLog	2018-02-13 19:03:47 UTC (rev 228425)
+++ tags/Safari-605.1.27.1/Source/WebCore/ChangeLog	2018-02-13 19:08:51 UTC (rev 228426)
@@ -1,3 +1,24 @@
+2018-02-12  Jason Marcell  <jmarc...@apple.com>
+
+        Apply patch. rdar://problem/37485750
+
+    CoreAudioCaptureSourceIOSListener should be a WebProcess singleton
+
+    2018-02-12  Youenn Fablet  <you...@apple.com>
+
+            CoreAudioCaptureSourceIOSListener should be a WebProcess singleton
+            https://bugs.webkit.org/show_bug.cgi?id=182606
+            <rdar://problem/37470882>
+
+            Reviewed by Eric Carlson.
+
+            Covered by manual testing.
+            In case shared unit is suspended, unsuspend it when creating a new audio source.
+            This creation should only happen when WebKit iOS is foregrounded.
+
+            * platform/mediastream/mac/CoreAudioCaptureSource.cpp:
+            (WebCore::CoreAudioCaptureSource::CoreAudioCaptureSource):
+
 2018-02-08  Jason Marcell  <jmarc...@apple.com>
 
         Cherry-pick r228258. rdar://problem/37355417

Modified: tags/Safari-605.1.27.1/Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp (228425 => 228426)


--- tags/Safari-605.1.27.1/Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp	2018-02-13 19:03:47 UTC (rev 228425)
+++ tags/Safari-605.1.27.1/Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp	2018-02-13 19:08:51 UTC (rev 228426)
@@ -724,6 +724,13 @@
     initializeVolume(unit.volume());
 
     unit.addClient(*this);
+
+#if PLATFORM(IOS)
+    // We ensure that we unsuspend ourselves on the constructor as a capture source
+    // is created when getUserMedia grants access which only happens when the process is foregrounded.
+    if (unit.isSuspended())
+        unit.reconfigureAudioUnit();
+#endif
 }
 
 CoreAudioCaptureSource::~CoreAudioCaptureSource()
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to