[Webkit-unassigned] [Bug 124265] merge MediaConstraintsImpl and MediaConstraints classes

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Nov 13 09:09:39 PST 2013


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





--- Comment #6 from Thiago de Barros Lacerda <thiago.lacerda at openbossa.org>  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.

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



More information about the webkit-unassigned mailing list