Minimize login prompts

chrome/browser/ui/login/login_prompt.h

 // Copyright (c) 2009 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 CHROME_BROWSER_UI_LOGIN_LOGIN_PROMPT_H_
 #define CHROME_BROWSER_UI_LOGIN_LOGIN_PROMPT_H_
 #pragma once
 
 #include <string>
 
@@ skipping 52 matching lines @@
   void OnRequestCancelled();
 
   // Implements the NotificationObserver interface.
   // Listens for AUTH_SUPPLIED and AUTH_CANCELLED notifications from other
   // LoginHandlers so that this LoginHandler has the chance to dismiss itself
   // if it was waiting for the same authentication.
   virtual void Observe(NotificationType type,
                        const NotificationSource& source,
                        const NotificationDetails& details);
 
+  // Who/where/what asked for the authentication.
+  const net::AuthChallengeInfo* auth_info() const { return auth_info_.get(); }
+
+  // Returns whether authentication had been handled (SetAuth or CancelAuth).
+  bool WasAuthHandled() const;
+
  protected:
   void SetModel(LoginModel* model);
 
   void SetDialog(ConstrainedWindow* dialog);
 
   // Notify observers that authentication is needed.
   void NotifyAuthNeeded();
 
   // Performs necessary cleanup before deletion.
   void ReleaseSoon();
 
-  // Who/where/what asked for the authentication.
-  net::AuthChallengeInfo* auth_info() const { return auth_info_.get(); }
-
  private:
   // Starts observing notifications from other LoginHandlers.
   void AddObservers();
 
   // Stops observing notifications from other LoginHandlers.
   void RemoveObservers();
 
   // Notify observers that authentication is supplied.
   void NotifyAuthSupplied(const std::wstring& username,
                           const std::wstring& password);
 
   // Notify observers that authentication is cancelled.
   void NotifyAuthCancelled();
 
-  // Returns whether authentication had been handled (SetAuth or CancelAuth).
-  // If |set_handled| is true, it will mark authentication as handled.
-  bool WasAuthHandled(bool set_handled);
+  // Marks authentication as handled and returns the previous handled
+  // state.
+  bool TestAndSetAuthHandled();
 
   // Calls SetAuth from the IO loop.
   void SetAuthDeferred(const std::wstring& username,
                        const std::wstring& password);
 
   // Calls CancelAuth from the IO loop.
   void CancelAuthDeferred();
 
   // Closes the view_contents from the UI loop.
   void CloseContentsDeferred();
 
   // True if we've handled auth (SetAuth or CancelAuth has been called).
   bool handled_auth_;
-  Lock handled_auth_lock_;
+  mutable Lock handled_auth_lock_;
 
   // The ConstrainedWindow that is hosting our LoginView.
   // This should only be accessed on the UI loop.
   ConstrainedWindow* dialog_;
 
   // Who/where/what asked for the authentication.
   scoped_refptr<net::AuthChallengeInfo> auth_info_;
 
   // The request that wants login data.
   // This should only be accessed on the IO loop.
@@ skipping 77 matching lines @@
 // Helper to remove the ref from an net::URLRequest to the LoginHandler.
 // Should only be called from the IO thread, since it accesses an
 // net::URLRequest.
 void ResetLoginHandlerForRequest(net::URLRequest* request);
 
 // Get the signon_realm under which the identity should be saved.
 std::string GetSignonRealm(const GURL& url,
                            const net::AuthChallengeInfo& auth_info);
 
 #endif  // CHROME_BROWSER_UI_LOGIN_LOGIN_PROMPT_H_