Bug 120875 - MediaStream API: Remove LocalMediaStream
Summary: MediaStream API: Remove LocalMediaStream
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:50 PDT by Eric Carlson
Modified: 2013-09-13 16:26 PDT (History)
18 users (show)

See Also:


Attachments
Patch (25.47 KB, patch)
2013-09-10 20:15 PDT, Thiago de Barros Lacerda
no flags Details | Formatted Diff | Diff
Patch (36.40 KB, patch)
2013-09-11 11:31 PDT, Thiago de Barros Lacerda
no flags Details | Formatted Diff | Diff
Patch (36.10 KB, patch)
2013-09-11 14:56 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:50:45 PDT
Consider merging https://chromium.googlesource.com/chromium/blink/+/81643ee3fe048a64fa8931132e1294fa2c8e821a 

The subclass LocalMediaStream have been removed from the getusermedia specification:
http://dev.w3.org/2011/webrtc/editor/getusermedia.html

The stop method will be moved temporarily to MediaStream until MediaStreamTrack has been updated.
Comment 1 Thiago de Barros Lacerda 2013-09-10 20:15:55 PDT
Created attachment 211275 [details]
Patch
Comment 2 Build Bot 2013-09-10 20:40:16 PDT
Comment on attachment 211275 [details]
Patch

Attachment 211275 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/1736748
Comment 3 Build Bot 2013-09-10 20:58:11 PDT
Comment on attachment 211275 [details]
Patch

Attachment 211275 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/1731829
Comment 4 kov's GTK+ EWS bot 2013-09-10 23:59:17 PDT
Comment on attachment 211275 [details]
Patch

Attachment 211275 [details] did not pass gtk-ews (gtk):
Output: http://webkit-queues.appspot.com/results/1743456
Comment 5 Thiago de Barros Lacerda 2013-09-11 11:31:24 PDT
Created attachment 211334 [details]
Patch
Comment 6 Eric Carlson 2013-09-11 13:51:58 PDT
Comment on attachment 211334 [details]
Patch

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

r=me with the changes noted.

> Source/WebCore/ChangeLog:8
> +        https://bugs.webkit.org/show_bug.cgi?id=120875
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        Existing tests were updated.

This should reference the Blink change this was merged from.

> Source/WebCore/Modules/mediastream/MediaStream.h:-53
> -    // DEPRECATED
> -    String label() const { return m_descriptor->id(); }
> -

This is unrelated, it should be removed in another patch.

> Source/WebCore/Modules/mediastream/MediaStream.idl:-37
> -    // DEPRECATED
> -    readonly attribute DOMString label;
> -

Ditto
Comment 7 Thiago de Barros Lacerda 2013-09-11 14:56:13 PDT
Created attachment 211356 [details]
Patch
Comment 8 WebKit Commit Bot 2013-09-11 15:31:29 PDT
Comment on attachment 211356 [details]
Patch

Clearing flags on attachment: 211356

Committed r155573: <http://trac.webkit.org/changeset/155573>