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

Unified Diff: remoting/jingle_glue/javascript_signal_strategy.cc

Issue 10378110: Verify that xmpp_login specified in the config matches auth token. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 7 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 | « remoting/jingle_glue/javascript_signal_strategy.h ('k') | remoting/jingle_glue/mock_objects.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/jingle_glue/javascript_signal_strategy.cc
diff --git a/remoting/jingle_glue/javascript_signal_strategy.cc b/remoting/jingle_glue/javascript_signal_strategy.cc
index 21cf3ccb339d6e278e104563a787fbb8895cf7fc..6703c52c28f3c776f9fe6f19f1741f4976c4b1c3 100644
--- a/remoting/jingle_glue/javascript_signal_strategy.cc
+++ b/remoting/jingle_glue/javascript_signal_strategy.cc
@@ -47,12 +47,21 @@ void JavascriptSignalStrategy::Disconnect() {
}
SignalStrategy::State JavascriptSignalStrategy::GetState() const {
+ DCHECK(CalledOnValidThread());
// TODO(sergeyu): Extend XmppProxy to provide status of the
// connection.
return CONNECTED;
}
+SignalStrategy::Error JavascriptSignalStrategy::GetError() const {
+ DCHECK(CalledOnValidThread());
+ // TODO(sergeyu): Extend XmppProxy to provide status of the
+ // connection.
+ return OK;
+}
+
std::string JavascriptSignalStrategy::GetLocalJid() const {
+ DCHECK(CalledOnValidThread());
return local_jid_;
}
« no previous file with comments | « remoting/jingle_glue/javascript_signal_strategy.h ('k') | remoting/jingle_glue/mock_objects.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698