Changeset 215834 in webkit

Timestamp:

Apr 26, 2017, 3:36:25 PM (7 years ago)

Author:

eric.carlson@apple.com

Message:

The current page is given a muted state when starting a media stream on iOS.
​https://bugs.webkit.org/show_bug.cgi?id=171290
<rdar://problem/31821427>

Reviewed by Jer Noble.

• UIProcess/UserMediaProcessManager.cpp:

(WebKit::UserMediaProcessManager::willEnableMediaStreamInPage): Don't mute the current
page on iOS either, that will prevent the new media stream from playing.

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/UserMediaProcessManager.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2017-04-26  Eric Carlson  <eric.carlson@apple.com>
+
+        The current page is given a muted state when starting a media stream on iOS.
+        https://bugs.webkit.org/show_bug.cgi?id=171290
+        <rdar://problem/31821427>
+
+        Reviewed by Jer Noble.
+
+        * UIProcess/UserMediaProcessManager.cpp:
+        (WebKit::UserMediaProcessManager::willEnableMediaStreamInPage): Don't mute the current
+        page on iOS either, that will prevent the new media stream from playing.
+
 2017-04-26  Alex Christensen  <achristensen@webkit.org>
 

trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.cpp

@@ -116 +116 @@
         for (auto& manager : state.value->managers()) {
 
-#if !PLATFORM(IOS)
             if (&manager->page() == &pageStartingCapture)
                 continue;
-#endif
 
             manager->page().setMuted(WebCore::MediaProducer::CaptureDevicesAreMuted);