Bug 66940 - [Chromium] MediaStream API: Add the MediaStreamRegistry interface
Summary: [Chromium] MediaStream API: Add the MediaStreamRegistry interface
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Tommy Widenflycht
URL:
Keywords:
Depends on:
Blocks: 56459
  Show dependency tree / graph
 
Reported: 2011-08-25 07:35 PDT by Tommy Widenflycht
Modified: 2011-12-19 11:08 PST (History)
4 users (show)

See Also:

Attachments
Patch (6.18 KB, patch)
2011-08-25 07:41 PDT, Tommy Widenflycht 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tommy Widenflycht 2011-08-25 07:35:03 PDT 
Implement the WebKit Chromium code providing the MediaStreamRegistry interface for the MediaStream API.
Comment 1 Tommy Widenflycht 2011-08-25 07:41:44 PDT 
Created attachment 105188 [details]
Patch
Comment 2 Darin Fisher (:fishd, Google) 2011-08-26 11:31:12 PDT 
Comment on attachment 105188 [details]
Patch

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

> Source/WebKit/chromium/public/WebMediaStreamRegistry.h:37
> +    static WebString mediaStreamLabel(const WebURL&);

Are you sure we don't want to create a WebMediaStream interface?
The WebKit API is designed to make it really easy to wrap ref-
counted WebCore types.  Then, WebMediaStream could have a label
property.

It would help if you could share your overall plans for MediaStream
APIs.  I don't have any idea what is coming next.
Comment 3 Eric Seidel (no email) 2011-12-19 11:08:09 PST 
Comment on attachment 105188 [details]
Patch

Cleared review? from attachment 105188 [details] so that this bug does not appear in http://webkit.org/pending-review.  If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).