<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:agold@apple.com" title="Andrew Gold <agold@apple.com>"> <span class="fn">Andrew Gold</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - Support for HTML Media Capture API"
href="https://bugs.webkit.org/show_bug.cgi?id=43239">bug 43239</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>agold@apple.com
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Support for HTML Media Capture API"
href="https://bugs.webkit.org/show_bug.cgi?id=43239#c8">Comment # 8</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Support for HTML Media Capture API"
href="https://bugs.webkit.org/show_bug.cgi?id=43239">bug 43239</a>
from <span class="vcard"><a class="email" href="mailto:agold@apple.com" title="Andrew Gold <agold@apple.com>"> <span class="fn">Andrew Gold</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=43239#c7">comment #7</a>)
<span class="quote">> Comment on <span class=""><a href="attachment.cgi?id=293962&action=diff" name="attach_293962" title="Proposed patch.">attachment 293962</a> <a href="attachment.cgi?id=293962&action=edit" title="Proposed patch.">[details]</a></span>
> Proposed patch.
>
> View in context:
> <a href="https://bugs.webkit.org/attachment.cgi?id=293962&action=review">https://bugs.webkit.org/attachment.cgi?id=293962&action=review</a>
>
> > Source/WebKit2/UIProcess/API/Cocoa/WKUIDelegatePrivate.h:67
> > +- (void)_webView:(WKWebView *)webVIew checkUserMediaPermissionForURL:(NSURL *)url mainFrameURL:(NSURL *)mainFrameURL frameIdentifier:(NSUInteger)frameIdentifier decisionHandler:(void (^)(NSString *salt, BOOL authorized))decisionHandler WK_API_AVAILABLE(ios(10.3));
>
> s/VI/Vi</span >
Done.
<span class="quote">>
> > Source/WebKit2/UIProcess/WKUserMediaProviderIOS.h:45
> > +@interface WKUserMediaProviderIOS : NSObject
>
> Do you think it would be reasonable to have the same interface on Mac, and
> call this WKUserMediaProvider, and have it implemented perhaps in platform
> specific ways (or perhaps shared!).</span >
Yes! It will have to be an interface and implemented differently on Mac and iOS because the AVFoundation APIs used in the class are platform specific, but I will make this change and implement it for iOS.
<span class="quote">>
> > Source/WebKit2/UIProcess/WKUserMediaProviderIOS.mm:56
> > +-(void)decidePolicyForUserMediaRequestFromOrigin:(SecurityOrigin&)userMediaOrigin topLevelOrigin:(SecurityOrigin&)topLevelOrigin frame:(WebFrameProxy&)frame request:(UserMediaPermissionRequestProxy&)permissionRequest view:(WKWebView*)view {
>
> Space after the -, and before the star in WKWebView*</span >
Done.
<span class="quote">>
> > Source/WebKit2/UIProcess/WebPageProxy.h:368
> > + PageClient& pageClient() { return m_pageClient; }
>
> This is surprising. The fact that it wasn't here before suggests that maybe
> it shouldn't be?</span >
Hm maybe not. I exposed it because the UserMediaPermissionRequestManagerProxy is calling decidePolicyForUserMediaPermissionRequest on the page proxy's uiClient for the C API so I wanted to do something similar for the WebKit2 API, but perhaps it would make more sense to have UserMediaPermissionRequestManagerProxy call decidePolicyForUserMediaPermissionRequest on the page proxy directly and the page proxy can call the appropriate methods on the respective clients.
Thanks for the review! V2 coming shortly.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>