Bug 120874 - MediaStream API: Enhance MediaStreamDescriptor add/remove component
Summary: MediaStream API: Enhance MediaStreamDescriptor add/remove component
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Thiago de Barros Lacerda
URL:
Keywords: BlinkMergeCandidate
Depends on:
Blocks: 121101
  Show dependency treegraph
 
Reported: 2013-09-06 11:49 PDT by Eric Carlson
Modified: 2013-09-10 10:30 PDT (History)
7 users (show)

See Also:


Attachments
Patch (6.41 KB, patch)
2013-09-09 10:36 PDT, Thiago de Barros Lacerda
no flags Details | Formatted Diff | Diff
Patch (7.58 KB, patch)
2013-09-09 15:01 PDT, Thiago de Barros Lacerda
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Carlson 2013-09-06 11:49:35 PDT
Consider merging https://chromium.googlesource.com/chromium/blink/+/67fcacf13ce922a762d7a1c6fb9e1b8e51e662ea

This patch allows the embedder to build up a MediaStreamDescriptor before announcing it to JS.
Also made a bit of refactoring to clean up the code in this area.
Comment 1 Thiago de Barros Lacerda 2013-09-09 10:36:01 PDT
Created attachment 211055 [details]
Patch
Comment 2 Eric Carlson 2013-09-09 10:50:55 PDT
Comment on attachment 211055 [details]
Patch

This is not a complete merge of the blink patch. It can't be completed until bug 120872 lands.
Comment 3 Thiago de Barros Lacerda 2013-09-09 15:01:33 PDT
Created attachment 211097 [details]
Patch
Comment 4 Thiago de Barros Lacerda 2013-09-09 16:07:22 PDT
Updating patch after https://bugs.webkit.org/show_bug.cgi?id=120872 and https://bugs.webkit.org/show_bug.cgi?id=121034 are landed
Comment 5 WebKit Commit Bot 2013-09-10 06:29:12 PDT
Comment on attachment 211097 [details]
Patch

Clearing flags on attachment: 211097

Committed r155434: <http://trac.webkit.org/changeset/155434>
Comment 6 WebKit Commit Bot 2013-09-10 06:29:14 PDT
All reviewed patches have been landed.  Closing bug.