Bug 112792

Summary: MediaStream API: Finalize the RTCPeerConnection states
Product: WebKit Reporter: Tommy Widenflycht <tommyw>
Component: WebCore Misc.Assignee: Tommy Widenflycht <tommyw>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dglazkov, eric.carlson, esprehn+autocc, feature-media-reviews, fishd, hta, jamesr, jer.noble, ojan.autocc, tkent+wkapi, webkit.review.bot
Priority: P2 Keywords: WebExposed
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 56459    
Attachments:
Description Flags
Patch none

Tommy Widenflycht
Reported 2013-03-20 05:49:52 PDT
MediaStream API: Finalize the RTCPeerConnection states
Attachments
Patch (18.40 KB, patch)
2013-03-20 05:56 PDT, Tommy Widenflycht 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Tommy Widenflycht
Comment 1 2013-03-20 05:56:26 PDT
Created attachment 194027 [details] Patch
Tommy Widenflycht
Comment 2 2013-03-20 05:58:58 PDT
This patch finalized the RTCPeerConnection state and events. These changes have not yet pushed out to the editors draft; but they are uncontroversial and agreed upon. readyState will be deleted; please use the replacement signalingState which is already available. onstatechange -> onsignalingstatechange onicechange -> oniceconnectionstatechange ongatheringchange will be deleted since oniceconnectionstatechange can be used for the same purpose. RTCIceConnectionState::"starting" -> RTCIceConnectionState::"new"
WebKit Review Bot
Comment 3 2013-03-20 06:00:07 PDT
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
WebKit Review Bot
Comment 4 2013-03-22 02:32:01 PDT
Comment on attachment 194027 [details] Patch Clearing flags on attachment: 194027 Committed r146582: <http://trac.webkit.org/changeset/146582>
WebKit Review Bot
Comment 5 2013-03-22 02:32:05 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.