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

Unified Diff: jingle/notifier/communicator/single_login_attempt.cc

Issue 2809056: Rewrote handing of auto-reconnection and network changes for sync notifier. (Closed)
Patch Set: Fixed bug Created 10 years, 5 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: jingle/notifier/communicator/single_login_attempt.cc
diff --git a/jingle/notifier/communicator/single_login_attempt.cc b/jingle/notifier/communicator/single_login_attempt.cc
index 8f1e26fee1f4cce5b03865d678ec2ce6bb61c01b..69dd21e9c3fc9b4c96950cdb24189d013d9e8886 100644
--- a/jingle/notifier/communicator/single_login_attempt.cc
+++ b/jingle/notifier/communicator/single_login_attempt.cc
@@ -99,10 +99,6 @@ SingleLoginAttempt::~SingleLoginAttempt() {
DCHECK(!client_);
}
-bool SingleLoginAttempt::auto_reconnect() const {
- return login_settings_->connection_options().auto_reconnect();
-}
-
const talk_base::ProxyInfo& SingleLoginAttempt::proxy() const {
DCHECK(connection_generator_.get());
return connection_generator_->proxy();
@@ -157,7 +153,7 @@ void SingleLoginAttempt::OnAttemptedAllConnections(
LOG(INFO) << "Connection failed with error " << code_;
// We were connected and we had a problem.
- if (successful_connection_ && auto_reconnect()) {
+ if (successful_connection_) {
SignalNeedAutoReconnect();
// Expect to be deleted at this point.
return;
« no previous file with comments | « jingle/notifier/communicator/single_login_attempt.h ('k') | jingle/notifier/listener/mediator_thread_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698