Bug 146560 - Creating bindings between WebCore, WebKit2 and Safari for getUserMedia
Summary: Creating bindings between WebCore, WebKit2 and Safari for getUserMedia
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Matthew Daiter
URL:
Keywords: HTML5, InRadar
Depends on:
Blocks:
 
Reported: 2015-07-02 15:33 PDT by Matthew Daiter
Modified: 2015-12-11 10:22 PST (History)
6 users (show)

See Also:


Attachments
Patch (11.31 KB, patch)
2015-07-02 15:45 PDT, Matthew Daiter
darin: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Daiter 2015-07-02 15:33:53 PDT
Needed to create a LOT of binding work in Safari and some binding in WebKit2 and WebCore between the three, in order to call getUserMedia
Comment 1 Radar WebKit Bug Importer 2015-07-02 15:39:56 PDT
<rdar://problem/21661498>
Comment 2 Matthew Daiter 2015-07-02 15:45:31 PDT
Created attachment 256042 [details]
Patch
Comment 3 Matthew Daiter 2015-07-02 15:48:26 PDT
(In reply to comment #0)
> Needed to create a LOT of binding work in Safari and some binding in WebKit2
> and WebCore between the three, in order to call getUserMedia

Clarification: in order to be able for getUserMedia to actually call into the browser, prompt the user for media access, and then go ahead and try to assess the promise if the constraints are correct.
Comment 4 Darin Adler 2015-07-08 12:33:38 PDT
Comment on attachment 256042 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=256042&action=review

> Source/WebKit2/ChangeLog:12
> +        (API::UIClient::decidePolicyForUserMediaPermissionRequest): Changed to pointers,
> +        like how Geolocation is doing it.

No, please don’t do that. New code should use references in cases like this.
Comment 5 Matthew Daiter 2015-07-10 10:57:00 PDT
Comment on attachment 256042 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=256042&action=review

>> Source/WebKit2/ChangeLog:12
>> +        like how Geolocation is doing it.
> 
> No, please don’t do that. New code should use references in cases like this.

Sure.
Comment 6 Brent Fulgham 2015-07-22 20:30:54 PDT
You need to fix the gtk build failure as well:

WebKitUIClient.cpp:170:18: error: 'virtual bool UIClient::decidePolicyForUserMediaPermissionRequest(WebKit::WebPageProxy&, WebKit::WebFrameProxy&, API::SecurityOrigin&, WebKit::UserMediaPermissionRequestProxy&)' marked override, but does not override