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

Side by Side Diff: jingle/notifier/communicator/single_login_attempt.cc

Issue 10545170: [Sync] Propagate XMPP auth errors to SyncNotifierObservers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix compile error Created 8 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <string> 5 #include <string>
6 6
7 #include "jingle/notifier/communicator/single_login_attempt.h" 7 #include "jingle/notifier/communicator/single_login_attempt.h"
8 8
9 #include "base/basictypes.h" 9 #include "base/basictypes.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
124 ServerInformation( 124 ServerInformation(
125 redirect_server, 125 redirect_server,
126 current_settings_->ssltcp_support)); 126 current_settings_->ssltcp_support));
127 // May be deleted at this point. 127 // May be deleted at this point.
128 return; 128 return;
129 } 129 }
130 } 130 }
131 } 131 }
132 } 132 }
133 133
134 if (error == buzz::XmppEngine::ERROR_UNAUTHORIZED) {
135 delegate_->OnRejectedCredentials();
136 return;
137 }
138
134 if (current_settings_ == settings_list_.end()) { 139 if (current_settings_ == settings_list_.end()) {
135 NOTREACHED(); 140 NOTREACHED();
136 return; 141 return;
137 } 142 }
138 143
139 ++current_settings_; 144 ++current_settings_;
140 if (current_settings_ == settings_list_.end()) { 145 if (current_settings_ == settings_list_.end()) {
141 VLOG(1) << "Could not connect to any XMPP server"; 146 VLOG(1) << "Could not connect to any XMPP server";
142 delegate_->OnNeedReconnect(); 147 delegate_->OnNeedReconnect();
143 return; 148 return;
(...skipping 18 matching lines...) Expand all
162 client_settings.token_service(), 167 client_settings.token_service(),
163 login_settings_.auth_mechanism()); 168 login_settings_.auth_mechanism());
164 xmpp_connection_.reset( 169 xmpp_connection_.reset(
165 new XmppConnection(client_settings, 170 new XmppConnection(client_settings,
166 login_settings_.request_context_getter(), 171 login_settings_.request_context_getter(),
167 this, 172 this,
168 pre_xmpp_auth)); 173 pre_xmpp_auth));
169 } 174 }
170 175
171 } // namespace notifier 176 } // namespace notifier
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698