Bug 112792 - MediaStream API: Finalize the RTCPeerConnection states
Summary: MediaStream API: Finalize the RTCPeerConnection states
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Tommy Widenflycht
URL:
Keywords: WebExposed
Depends on:
Blocks: 56459
  Show dependency tree / graph
 
Reported: 2013-03-20 05:49 PDT by Tommy Widenflycht
Modified: 2013-03-22 02:32 PDT (History)
12 users (show)

See Also:

Attachments
Patch (18.40 KB, patch)
2013-03-20 05:56 PDT, Tommy Widenflycht 		no flags Details | Formatted Diff | Diff
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 2013-03-20 05:49:52 PDT 
MediaStream API: Finalize the RTCPeerConnection states
Comment 1 Tommy Widenflycht 2013-03-20 05:56:26 PDT 
Created attachment 194027 [details]
Patch
Comment 2 Tommy Widenflycht 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"
Comment 3 WebKit Review Bot 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.
Comment 4 WebKit Review Bot 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>
Comment 5 WebKit Review Bot 2013-03-22 02:32:05 PDT 
All reviewed patches have been landed.  Closing bug.