[Sync] Propagate XMPP auth errors to SyncNotifierObservers

jingle/notifier/communicator/login.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef JINGLE_NOTIFIER_COMMUNICATOR_LOGIN_H_
 #define JINGLE_NOTIFIER_COMMUNICATOR_LOGIN_H_
 
 #include <string>
 
 #include "base/compiler_specific.h"
@@ skipping 15 matching lines @@
 }  // namespace buzz
 
 namespace net {
 class URLRequestContextGetter;
 }  // namespace net
 
 namespace notifier {
 
 class LoginSettings;
 
-// Workaround for MSVS 2005 bug that fails to handle inheritance from a nested
-// class properly if it comes directly on a base class list.
-typedef SingleLoginAttempt::Delegate SingleLoginAttemptDelegate;
-
-// Does the login, keeps it alive (with refreshing cookies and reattempting
-// login when disconnected), figures out what actions to take on the various
-// errors that may occur.
+// Does the login, keeps it alive (with refreshing cookies and
+// reattempting login when disconnected), and figures out what actions
+// to take on the various errors that may occur.
 class Login : public net::NetworkChangeNotifier::IPAddressObserver,
-              public SingleLoginAttemptDelegate {
+              public SingleLoginAttempt::Delegate {
  public:
   class Delegate {
    public:
+    // Called when a connection has been successfully established.
     virtual void OnConnect(
         base::WeakPtr<buzz::XmppTaskParentInterface> base_task) = 0;
+
+    // Called when we've detected a disconnection.  A reconnection
+    // will be attempted.  May be called multiple times in a row.
     virtual void OnDisconnect() = 0;
 
+    // Called when our credentials have been rejected.  Will not try
+    // to reconnect after this (although the client may still call
+    // StartConnection(), perhaps after calling UpdateXmppSettings()).
+    virtual void OnRejectedCredentials() = 0;
+
    protected:
-    virtual ~Delegate() {}
+    virtual ~Delegate();
   };
 
   // Does not take ownership of |delegate|, which must not be NULL.
   Login(Delegate* delegate,
         const buzz::XmppClientSettings& user_settings,
         const scoped_refptr<net::URLRequestContextGetter>&
             request_context_getter,
         const ServerList& servers,
         bool try_ssltcp_first,
         const std::string& auth_mechanism);
   virtual ~Login();
 
+  // Starts connecting (or forces a reconnection if we're backed off).
   void StartConnection();
-  // The updated settings only take effect the next time StartConnection
-  // is called.
+
+  // The updated settings take effect only the next time when a
+  // connection is attempted (either via reconnection or a call to
+  // StartConnection()).
   void UpdateXmppSettings(const buzz::XmppClientSettings& user_settings);
 
   // net::NetworkChangeNotifier::IPAddressObserver implementation.
   virtual void OnIPAddressChanged() OVERRIDE;
 
   // SingleLoginAttempt::Delegate implementation.
   virtual void OnConnect(
       base::WeakPtr<buzz::XmppTaskParentInterface> base_task) OVERRIDE;
   virtual void OnNeedReconnect() OVERRIDE;
   virtual void OnRedirect(const ServerInformation& redirect_server) OVERRIDE;
+  virtual void OnRejectedCredentials() OVERRIDE;
 
  private:
   void OnLogoff();
 
   // Stops any existing reconnect timer and sets an initial reconnect
   // interval.
   void ResetReconnectState();
 
   // Tries to reconnect in some point in the future.  If called
   // repeatedly, will wait longer and longer until reconnecting.
   void TryReconnect();
 
   // The actual function (called by |reconnect_timer_|) that does the
   // reconnection.
   void DoReconnect();
 
   Delegate* const delegate_;
   LoginSettings login_settings_;
   scoped_ptr<SingleLoginAttempt> single_attempt_;
 
   // reconnection state.
   base::TimeDelta reconnect_interval_;
   base::OneShotTimer<Login> reconnect_timer_;
 
   DISALLOW_COPY_AND_ASSIGN(Login);
 };
 
-// Workaround for MSVS 2005 bug that fails to handle inheritance from a nested
-// class properly if it comes directly on a base class list.
-typedef Login::Delegate LoginDelegate;
-
 }  // namespace notifier
 
 #endif  // JINGLE_NOTIFIER_COMMUNICATOR_LOGIN_H_