[Webkit-unassigned] [Bug 68460] Add WebCore platform interfaces needed by updated MediaStream and PeerConnection design

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=68460





--- Comment #14 from Adam Barth <abarth at webkit.org>  2011-09-22 08:11:40 PST ---
I think there's some miscommunication here, so let me start from the beginning.  My sense is that we've got two somewhat competing approaches to implementing this feature.  On the one hand, there's this patch and, on the other hand, there's https://bugs.webkit.org/attachment.cgi?id=105149&action=review.

Neither of these approaches seems like exactly what we want, but they both have good aspects to them that hopefully we can combine.  To start with, I like your approach of having a Platform layer and a WebCore/platform/mediastream directory.  The main question, as Alexey points out, is what API the Platform layer should expose to the rest of WebCore.  Ideally, we'd expose a simple API that could be implemented by multiple low-level libraries (e.g., that handle ICE and the various other RTC-specific bits).

Tommy's patch has a relatively simple API that I'd be interested in whether you think would work here.  (In his patch, he's put the API at the Client layer, but putting it at the Platform layer seems more appropriate.)  Specifically, consider the API implied by this object:

http://trac.webkit.org/browser/trunk/Source/WebCore/page/MediaStreamClient.h

One thing we should change is to replace the various IDs with actual objects, but aside from that, these seem like the basic operations (and an appropriate level of detail) for the Platform to expose.

How would feel about moving MediaStreamClient from being a Client interface to being a Platform interface?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.