Attachment 340107 Details for Bug 185516 – Patch

[patch] Patch

bug-185516-20180510105817.patch (text/plain), 3.32 KB, created by Eric Carlson on 2018-05-10 10:58:18 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Eric Carlson

Created: 2018-05-10 10:58:18 PDT

Size: 3.32 KB

patch

obsolete

>Subversion Revision: 231643
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 90ca7af004f6cde3f5acc81490d1e2893f71aac9..1c88fbe50e2f8cd0941c38d5136437d6efbf82f6 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,15 @@
>+2018-05-10  Eric Carlson  <eric.carlson@apple.com>
>+
>+        [MediaStream, iOS] Don't check authorizationStatusForMediaType when using mock capture devices
>+        https://bugs.webkit.org/show_bug.cgi?id=185516
>+        <rdar://problem/36328191>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * UIProcess/Cocoa/UIDelegate.mm:
>+        (WebKit::UIDelegate::UIClient::decidePolicyForUserMediaPermissionRequest): Don't check
>+        +[AVCaptureDevice authorizationStatusForMediaType:] when using mock capture devices.
>+
> 2018-05-09  Carlos Garcia Campos  <cgarcia@igalia.com>
> 
>         WebDriver: implement advance user interactions
>diff --git a/Source/WebKit/UIProcess/Cocoa/UIDelegate.mm b/Source/WebKit/UIProcess/Cocoa/UIDelegate.mm
>index 0c0477c22ac35da2e6e211185f050c4080b0386d..69c45dff3385432a7fd6aa07307e5ff1d94055a9 100644
>--- a/Source/WebKit/UIProcess/Cocoa/UIDelegate.mm
>+++ b/Source/WebKit/UIProcess/Cocoa/UIDelegate.mm
>@@ -884,15 +884,16 @@ bool UIDelegate::UIClient::decidePolicyForUserMediaPermissionRequest(WebPageProx
>         request.deny(UserMediaPermissionRequestProxy::UserMediaAccessDenialReason::NoConstraints);
>         return true;
>     }
>+    bool usingMockCaptureDevices = page.preferences().mockCaptureDevicesEnabled();
> 
> #if PLATFORM(IOS)
>-    auto requestCameraAuthorization = BlockPtr<void()>::fromCallable([this, &frame, protectedRequest = makeRef(request), webView = RetainPtr<WKWebView>(m_uiDelegate.m_webView)]() {
>+    auto requestCameraAuthorization = BlockPtr<void()>::fromCallable([this, &frame, protectedRequest = makeRef(request), webView = RetainPtr<WKWebView>(m_uiDelegate.m_webView), usingMockCaptureDevices]() {
> 
>         if (!protectedRequest->requiresVideoCapture()) {
>             requestUserMediaAuthorizationForDevices(frame, protectedRequest, (id <WKUIDelegatePrivate>)m_uiDelegate.m_delegate.get(), *webView.get());
>             return;
>         }
>-        AVAuthorizationStatus cameraAuthorizationStatus = [getAVCaptureDeviceClass() authorizationStatusForMediaType:getAVMediaTypeVideo()];
>+        AVAuthorizationStatus cameraAuthorizationStatus = usingMockCaptureDevices ? AVAuthorizationStatusAuthorized : [getAVCaptureDeviceClass() authorizationStatusForMediaType:getAVMediaTypeVideo()];
>         switch (cameraAuthorizationStatus) {
>         case AVAuthorizationStatusAuthorized:
>             requestUserMediaAuthorizationForDevices(frame, protectedRequest, (id <WKUIDelegatePrivate>)m_uiDelegate.m_delegate.get(), *webView.get());
>@@ -916,7 +917,7 @@ bool UIDelegate::UIClient::decidePolicyForUserMediaPermissionRequest(WebPageProx
>     });
> 
>     if (requiresAudioCapture) {
>-        AVAuthorizationStatus microphoneAuthorizationStatus = [getAVCaptureDeviceClass() authorizationStatusForMediaType:getAVMediaTypeAudio()];
>+        AVAuthorizationStatus microphoneAuthorizationStatus = usingMockCaptureDevices ? AVAuthorizationStatusAuthorized : [getAVCaptureDeviceClass() authorizationStatusForMediaType:getAVMediaTypeAudio()];
>         switch (microphoneAuthorizationStatus) {
>         case AVAuthorizationStatusAuthorized:
>             requestCameraAuthorization();

Actions: View | Formatted Diff | Diff

Attachments on bug 185516: 340107 | 340109