RESOLVED INVALID 124265
Move MediaConstraintsImpl to MediaConstraintsPrivate in platform/
https://bugs.webkit.org/show_bug.cgi?id=124265
Summary Move MediaConstraintsImpl to MediaConstraintsPrivate in platform/
Philippe Normand
Reported 2013-11-12 23:15:58 PST
The Impl class is a remmnant of the Chromium days.
Attachments
patch (19.73 KB, patch)
2013-11-13 00:34 PST, Philippe Normand
no flags
Philippe Normand
Comment 1 2013-11-13 00:34:30 PST
Created attachment 216776 [details] patch Doesn't yet include XCode project changes.
Build Bot
Comment 2 2013-11-13 01:04:14 PST
Build Bot
Comment 3 2013-11-13 01:31:32 PST
Thiago de Barros Lacerda
Comment 4 2013-11-13 06:10:13 PST
+1 for this change. There is no reason to have an impl class if there will not have platform dependent implementation of that
Eric Carlson
Comment 5 2013-11-13 08:56:41 PST
It is a layering violation to pass an object defined in Modules/mediastream to code in platform. What will we use in platform, for example what gets passed to MediaStreamCenter::validateRequestConstraints?
Thiago de Barros Lacerda
Comment 6 2013-11-13 09:08:18 PST
(In reply to comment #5) > It is a layering violation to pass an object defined in Modules/mediastream to code in platform. > > What will we use in platform, for example what gets passed to MediaStreamCenter::validateRequestConstraints? You are right. I don't know why, but I always thought that the platform implementation was the Impl one, but in this case is MediaConstraints. But indeed, we have to leave a platform implementation. We could leave MediaConstraints in Modules/mediastream, and create a MediaConstraintsPrivate that would be in platform. To summarize, would only be a rename of classes.
Philippe Normand
Comment 7 2014-08-25 05:06:34 PDT
Comment on attachment 216776 [details] patch Ok let's finally do this properly then :)
Philippe Normand
Comment 8 2014-08-26 09:04:13 PDT
Well, I'm not sure anymore this is a real issue, if we move the Impl class to platform/ and rename it to Private it's not exactly like other Private implementations we have in platform/mediastream. MediaConstraintsImpl is the actual implementation of an abstract class whereas other Private classes don't inherit from the interface they implement, it's a different pattern... For bug 123158 I got a PoC that is able to send MediaConstraintsImpl over the wire but I'm not sure if this is a layer violation or not. I'd prefer to handle MediaConstraints directly but being an abstract class I'm having issues for the IPC communication.
Jon Lee
Comment 9 2017-08-24 16:36:01 PDT
This has been removed via 172132.
Note You need to log in before you can comment on or make changes to this bug.