This patch adds readyState functionality to RTCPeerConnection, together with related callbacks and methods.
Created attachment 160160 [details] Patch
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 on attachment 160160 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=160160&action=review > Source/WebCore/platform/mediastream/RTCPeerConnectionHandlerClient.h:42 > + virtual void didChangeReadyState(uint32_t state) = 0; uint32_t -> ReadyState Perhaps we need to move the ReadyState enum to a more general location so we can reference it here?
Looks like you've got a compile error on GTK to work through as well.
Created attachment 160402 [details] Patch
Fixed. (In reply to comment #4) > Looks like you've got a compile error on GTK to work through as well.
Comment on attachment 160160 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=160160&action=review >> Source/WebCore/platform/mediastream/RTCPeerConnectionHandlerClient.h:42 >> + virtual void didChangeReadyState(uint32_t state) = 0; > > uint32_t -> ReadyState > > Perhaps we need to move the ReadyState enum to a more general location so we can reference it here? Fixed by moving the ReadyState definition to this class.
Comment on attachment 160402 [details] Patch Clearing flags on attachment: 160402 Committed r126586: <http://trac.webkit.org/changeset/126586>
All reviewed patches have been landed. Closing bug.