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

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: Merging with master (MediaStreamSignaling removal affected WebRtcSession). 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
« no previous file with comments | « talk/app/webrtc/peerconnection.h ('k') | talk/app/webrtc/statscollector.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: talk/app/webrtc/peerconnection.cc
diff --git a/talk/app/webrtc/peerconnection.cc b/talk/app/webrtc/peerconnection.cc
index 4df6790a1a1fa9c4429e9043b5edd07397c4da46..e7b33c43acdbd917ad549aac93df2611fd818475 100644
--- a/talk/app/webrtc/peerconnection.cc
+++ b/talk/app/webrtc/peerconnection.cc
@@ -1163,32 +1163,31 @@ void PeerConnection::Close() {
// streams before the channels are closed.
stats_->UpdateStats(kStatsOutputLevelStandard);
- session_->Terminate();
+ session_->Close();
}
-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_CLOSED:
ChangeSignalingState(PeerConnectionInterface::kClosed);
break;
default:
« no previous file with comments | « talk/app/webrtc/peerconnection.h ('k') | talk/app/webrtc/statscollector.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698