Chromium Code Reviews| Index: content/renderer/media/rtc_peer_connection_handler.cc |
| =================================================================== |
| --- content/renderer/media/rtc_peer_connection_handler.cc (revision 176188) |
| +++ content/renderer/media/rtc_peer_connection_handler.cc (working copy) |
| @@ -62,7 +62,7 @@ |
| static WebKit::WebRTCPeerConnectionHandlerClient::ReadyState |
| GetWebKitReadyState(webrtc::PeerConnectionInterface::ReadyState ready_state) { |
| switch (ready_state) { |
| - case webrtc::PeerConnectionInterface::kNew: |
| + case webrtc::PeerConnectionInterface::kStable: |
| return WebKit::WebRTCPeerConnectionHandlerClient::ReadyStateNew; |
|
Ronghua Wu (Left Chromium)
2013/01/11 00:52:32
Per, what should we return here since the WebRTCPe
perkj_chrome
2013/01/11 07:57:01
Please check later today if Tommy Ws cl for adding
Ronghua Wu (Left Chromium)
2013/01/11 19:03:01
Done.
|
| case webrtc::PeerConnectionInterface::kHaveLocalOffer: |
| @@ -70,8 +70,6 @@ |
| case webrtc::PeerConnectionInterface::kHaveRemoteOffer: |
| case webrtc::PeerConnectionInterface::kHaveRemotePrAnswer: |
| return WebKit::WebRTCPeerConnectionHandlerClient::ReadyStateOpening; |
| - case webrtc::PeerConnectionInterface::kActive: |
| - return WebKit::WebRTCPeerConnectionHandlerClient::ReadyStateActive; |
| case webrtc::PeerConnectionInterface::kClosed: |
| return WebKit::WebRTCPeerConnectionHandlerClient::ReadyStateClosed; |
| default: |
| @@ -517,7 +515,7 @@ |
| void RTCPeerConnectionHandler::OnStateChange(StateType state_changed) { |
| switch (state_changed) { |
| - case kReadyState: { |
| + case kSignalingState: { |
| WebKit::WebRTCPeerConnectionHandlerClient::ReadyState ready_state = |
| GetWebKitReadyState(native_peer_connection_->ready_state()); |
| client_->didChangeReadyState(ready_state); |