Bug 120874

Summary: MediaStream API: Enhance MediaStreamDescriptor add/remove component
Product: WebKit Reporter: Eric Carlson <eric.carlson>
Component: WebCore Misc.Assignee: Thiago de Barros Lacerda <thiago.lacerda>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, eric.carlson, glenn, hta, jer.noble, luciano.wolf, tommyw
Priority: P2 Keywords: BlinkMergeCandidate
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 121101    
Attachments:
Description Flags
Patch
none
Patch none

Eric Carlson
Reported 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.
Attachments
Patch (6.41 KB, patch)
2013-09-09 10:36 PDT, Thiago de Barros Lacerda 		no flags
DetailsFormatted DiffDiff
Patch (7.58 KB, patch)
2013-09-09 15:01 PDT, Thiago de Barros Lacerda 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Thiago de Barros Lacerda
Comment 1 2013-09-09 10:36:01 PDT
Created attachment 211055 [details] Patch
Eric Carlson
Comment 2 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.
Thiago de Barros Lacerda
Comment 3 2013-09-09 15:01:33 PDT
Created attachment 211097 [details] Patch
Thiago de Barros Lacerda
Comment 4 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
WebKit Commit Bot
Comment 5 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>
WebKit Commit Bot
Comment 6 2013-09-10 06:29:14 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.