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

Unified Diff: remoting/protocol/jingle_session.cc

Issue 2798393007: Use SignalingAddress in SignalStrategy insterface. (Closed)
Patch Set: . Created 3 years, 8 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: remoting/protocol/jingle_session.cc
diff --git a/remoting/protocol/jingle_session.cc b/remoting/protocol/jingle_session.cc
index b56b8a5bf514639d61d798e37b295d1454322b39..c05d1a3dbb926f32d20e0ee8d9af260d3ae63a7d 100644
--- a/remoting/protocol/jingle_session.cc
+++ b/remoting/protocol/jingle_session.cc
@@ -197,13 +197,13 @@ ErrorCode JingleSession::error() {
}
void JingleSession::StartConnection(
- const std::string& peer_jid,
+ const SignalingAddress& peer_address,
std::unique_ptr<Authenticator> authenticator) {
DCHECK(thread_checker_.CalledOnValidThread());
DCHECK(authenticator.get());
DCHECK_EQ(authenticator->state(), Authenticator::MESSAGE_READY);
- peer_address_ = SignalingAddress(peer_jid);
+ peer_address_ = peer_address;
authenticator_ = std::move(authenticator);
// Generate random session ID. There are usually not more than 1
@@ -792,7 +792,8 @@ void JingleSession::SendSessionInitiateMessage() {
}
std::unique_ptr<JingleMessage> message(new JingleMessage(
peer_address_, JingleMessage::SESSION_INITIATE, session_id_));
- message->initiator = session_manager_->signal_strategy_->GetLocalJid();
+ message->initiator =
+ session_manager_->signal_strategy_->GetLocalAddress().jid();
message->description.reset(new ContentDescription(
session_manager_->protocol_config_->Clone(),
authenticator_->GetNextMessage()));

Powered by Google App Engine
This is Rietveld 408576698