Bug 71344

Summary: Remove initMediaStreamEvent method
Product: WebKit Reporter: Dominic Cooney <dominicc>
Component: DOMAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, kaustubh.ra, ojan, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 71345    
Bug Blocks: 68791    
Attachments:
Description Flags
Patch none

Dominic Cooney
Reported 2011-11-01 18:08:51 PDT
This method has been removed from the spec draft: <http://www.whatwg.org/specs/web-apps/current-work/multipage/video-conferencing-and-peer-to-peer-communication.html#mediastreamevent> Code Search does not turn up any users: <http://codesearch.google.com/#search/&q=initMediaStreamEvent&type=cs> Since this is new, it is probably safe to remove it when MediaStreamEvent gets a constructor.
Attachments
Patch (2.99 KB, patch)
2011-11-04 05:19 PDT, Kaustubh Atrawalkar 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dominic Cooney
Comment 1 2011-11-01 18:12:58 PDT
Bug 71345 is tracking implementing the MediaStreamEvent constructor.
Kaustubh Atrawalkar
Comment 2 2011-11-04 05:19:23 PDT
Created attachment 113649 [details] Patch
Kaustubh Atrawalkar
Comment 3 2011-11-04 05:19:51 PDT
Added patch for removing initMediaStreamEvent method.
Dominic Cooney
Comment 4 2011-11-04 09:13:29 PDT
Comment on attachment 113649 [details] Patch The patch looks good to me. You may want to hold off landing this until the MediaStreamEvent constructor is implemented (see bug 71345) otherwise there will be no way to programatically dispatch a MediaStreamEvent.
Adam Barth
Comment 5 2011-11-04 10:03:43 PDT
Comment on attachment 113649 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=113649&action=review I think its fine to land without the constructor. This feature isn't shipping in any port that I'm aware of. > Source/WebCore/ChangeLog:11 > + No new tests. Removed method. You can test that this method doesn't exist! That will catch us if someone tries to add it back.
WebKit Review Bot
Comment 6 2011-11-04 11:12:02 PDT
Comment on attachment 113649 [details] Patch Clearing flags on attachment: 113649 Committed r99299: <http://trac.webkit.org/changeset/99299>
WebKit Review Bot
Comment 7 2011-11-04 11:12:07 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.