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

Unified Diff: remoting/host/host_change_notification_listener.cc

Issue 2798393007: Use SignalingAddress in SignalStrategy insterface. (Closed)
Patch Set: header 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/host/host_change_notification_listener.cc
diff --git a/remoting/host/host_change_notification_listener.cc b/remoting/host/host_change_notification_listener.cc
index 3c079f19062adc8df1727247605aaebe89e9ea8c..c80af519e5dbf567f3ad3d95555ef09796b2b338 100644
--- a/remoting/host/host_change_notification_listener.cc
+++ b/remoting/host/host_change_notification_listener.cc
@@ -55,9 +55,13 @@ bool HostChangeNotificationListener::OnSignalStrategyIncomingStanza(
const std::string& host_id =
host_changed_element->Attr(QName(kChromotingXmlNamespace, "hostid"));
const std::string& from = stanza->Attr(buzz::QN_FROM);
- const std::string& to = stanza->Attr(buzz::QN_TO);
+
+ std::string to_error;
+ SignalingAddress to =
+ SignalingAddress::Parse(stanza, SignalingAddress::TO, &to_error);
+
if (host_id == host_id_ && from == directory_bot_jid_ &&
- to == signal_strategy_->GetLocalJid()) {
+ to == signal_strategy_->GetLocalAddress()) {
const std::string& operation =
host_changed_element->Attr(QName(kChromotingXmlNamespace, "operation"));
if (operation == "delete") {
« no previous file with comments | « remoting/host/heartbeat_sender_unittest.cc ('k') | remoting/host/host_change_notification_listener_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698