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

Unified Diff: remoting/protocol/jingle_session.h

Issue 8743023: Separate Authenticator and Session unittests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge Created 9 years 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: remoting/protocol/jingle_session.h
diff --git a/remoting/protocol/jingle_session.h b/remoting/protocol/jingle_session.h
index 75f061fec8d0cf57d3b563b4a5f050f149796525..4894216d9bf788dde3f50a8878abd6a28d671611 100644
--- a/remoting/protocol/jingle_session.h
+++ b/remoting/protocol/jingle_session.h
@@ -80,6 +80,8 @@ class JingleSession : public protocol::Session,
// Used for Session.SignalError sigslot.
void OnSessionError(cricket::BaseSession* session,
cricket::BaseSession::Error error);
+ // Used for Session.SignalReceivedTerminateReason sigslot.
+ void OnTerminateReason(cricket::Session* session, const std::string& reason);
void OnInitiate();
void OnAccept();
@@ -135,6 +137,11 @@ class JingleSession : public protocol::Session,
// Channels that are currently being connected.
ChannelConnectorsMap channel_connectors_;
+ // Termination reason. Needs to be stored because
+ // SignalReceivedTerminateReason handler is not allowed to destroy
+ // the object.
+ std::string terminate_reason_;
+
ScopedRunnableMethodFactory<JingleSession> task_factory_;
DISALLOW_COPY_AND_ASSIGN(JingleSession);

Powered by Google App Engine
This is Rietveld 408576698