Bug 60394 - [GTK] Enable Media Stream feature and make it default on
Summary: [GTK] Enable Media Stream feature and make it default on
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 60174 60391
Blocks:
  Show dependency treegraph
 
Reported: 2011-05-06 12:14 PDT by Adam Bergkvist
Modified: 2011-06-19 20:06 PDT (History)
3 users (show)

See Also:


Attachments
Proposed patch (5.94 KB, patch)
2011-05-06 12:27 PDT, Adam Bergkvist
no flags Details | Formatted Diff | Diff
Updated patch (5.89 KB, patch)
2011-05-11 02:52 PDT, Adam Bergkvist
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Bergkvist 2011-05-06 12:14:53 PDT
Enable the Media Stream feature and unskip the tests.
Comment 1 Adam Bergkvist 2011-05-06 12:27:48 PDT
Created attachment 92623 [details]
Proposed patch
Comment 2 Martin Robinson 2011-05-06 12:32:29 PDT
It'd be easier to review this patch if you provided a bit of context about the media stream feature and why it's good to have it default to enabled. :)
Comment 3 Adam Bergkvist 2011-05-09 02:16:15 PDT
(In reply to comment #2)
> It'd be easier to review this patch if you provided a bit of context about the media stream feature and why it's good to have it default to enabled. :)

We are implementing "chapter 9" in the WHATWG HTML specification - Video conferencing and peer-to-peer communication (http://www.whatwg.org/specs/web-apps/current-work/multipage/dnd.html#video-conferencing-and-peer-to-peer-communication). Other people seem to be doing some work in this area as well and we want to make sure that whatever code that is added works on GTK and passes the tests. Thus we would like to enable this feature by default to have the build bots run the tests.
Comment 4 Martin Robinson 2011-05-09 09:35:15 PDT
Is this feature complete or experimental? If it's experimental and you want to run tests on the bots, we should at least default it to off in configure.ac. That will ensure that people compiling our releases, won't be automatically saddled with an incomplete, perhaps buggy feature.
Comment 5 Adam Bergkvist 2011-05-11 02:52:32 PDT
Created attachment 93097 [details]
Updated patch

Yes you're right. This feature is incomplete and experimental and should be turned off by default in configure.ac.
Comment 6 Martin Robinson 2011-05-11 09:16:15 PDT
Comment on attachment 93097 [details]
Updated patch

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

> configure.ac:573
> +                             [enable media stream support (incomplete) [default=no]]),

When landing please change this to say "(experimental)"  here to match the other flags.
Comment 7 Eric Seidel (no email) 2011-06-18 20:01:31 PDT
Adam is not a committer, so unless you're asking him to upload a new patch Martin, we should either r- this or cq+ it.
Comment 8 Martin Robinson 2011-06-19 16:31:53 PDT
(In reply to comment #7)
> Adam is not a committer, so unless you're asking him to upload a new patch Martin, we should either r- this or cq+ it.

I think that incomplete is probably fine here in place of experimental.
Comment 9 WebKit Review Bot 2011-06-19 20:06:03 PDT
Comment on attachment 93097 [details]
Updated patch

Clearing flags on attachment: 93097

Committed r89227: <http://trac.webkit.org/changeset/89227>
Comment 10 WebKit Review Bot 2011-06-19 20:06:08 PDT
All reviewed patches have been landed.  Closing bug.