Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(520)

Unified Diff: talk/app/webrtc/peerconnection.cc

Issue 1397973002: Merging BaseSession code into WebRtcSession. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Cleaning up WebRtcSession states, and getting rid of "saved candidates" Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: talk/app/webrtc/peerconnection.cc
diff --git a/talk/app/webrtc/peerconnection.cc b/talk/app/webrtc/peerconnection.cc
index 86902b03591277b91d4a6106b38e7caf8e04d495..4a4cd8c31679dcce232fb6e08c9f4e7a160a37ff 100644
--- a/talk/app/webrtc/peerconnection.cc
+++ b/talk/app/webrtc/peerconnection.cc
@@ -786,29 +786,28 @@ void PeerConnection::Close() {
session_->Terminate();
}
-void PeerConnection::OnSessionStateChange(cricket::BaseSession* /*session*/,
- cricket::BaseSession::State state) {
+void PeerConnection::OnSessionStateChange(WebRtcSession* /*session*/,
+ WebRtcSession::State state) {
switch (state) {
- case cricket::BaseSession::STATE_INIT:
+ case WebRtcSession::STATE_INIT:
ChangeSignalingState(PeerConnectionInterface::kStable);
break;
- case cricket::BaseSession::STATE_SENTINITIATE:
+ case WebRtcSession::STATE_SENTOFFER:
ChangeSignalingState(PeerConnectionInterface::kHaveLocalOffer);
break;
- case cricket::BaseSession::STATE_SENTPRACCEPT:
+ case WebRtcSession::STATE_SENTPRANSWER:
ChangeSignalingState(PeerConnectionInterface::kHaveLocalPrAnswer);
break;
- case cricket::BaseSession::STATE_RECEIVEDINITIATE:
+ case WebRtcSession::STATE_RECEIVEDOFFER:
ChangeSignalingState(PeerConnectionInterface::kHaveRemoteOffer);
break;
- case cricket::BaseSession::STATE_RECEIVEDPRACCEPT:
+ case WebRtcSession::STATE_RECEIVEDPRANSWER:
ChangeSignalingState(PeerConnectionInterface::kHaveRemotePrAnswer);
break;
- case cricket::BaseSession::STATE_SENTACCEPT:
- case cricket::BaseSession::STATE_RECEIVEDACCEPT:
+ case WebRtcSession::STATE_INPROGRESS:
ChangeSignalingState(PeerConnectionInterface::kStable);
break;
- case cricket::BaseSession::STATE_RECEIVEDTERMINATE:
+ case WebRtcSession::STATE_TERMINATED:
pthatcher1 2015/10/10 00:07:26 STATE_CLOSED?
Taylor Brandstetter 2015/10/10 00:30:15 Done.
ChangeSignalingState(PeerConnectionInterface::kClosed);
break;
default:

Powered by Google App Engine
This is Rietveld 408576698