Add auto-refresh support to OAuthTokenGetter (disabled for now).

remoting/host/oauth_token_getter.h

 // Copyright 2014 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 REMOTING_HOST_OAUTH_TOKEN_GETTER_H_
 #define REMOTING_HOST_OAUTH_TOKEN_GETTER_H_
 
 #include <queue>
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/threading/non_thread_safe.h"
+#include "base/time/time.h"
+#include "base/timer/timer.h"
 #include "google_apis/gaia/gaia_oauth_client.h"
 
 namespace net {
 class URLRequestContextGetter;
 }  // namespace net
 
 namespace remoting {
 
 // OAuthTokenGetter caches OAuth access tokens and refreshes them as needed.
 class OAuthTokenGetter :
@@ skipping 26 matching lines @@
 
     // Token delegating authority to us to act as the user.
     std::string refresh_token;
 
     // Whether these credentials belong to a service account.
     bool is_service_account;
   };
 
   OAuthTokenGetter(
       scoped_ptr<OAuthCredentials> oauth_credentials,
-      scoped_refptr<net::URLRequestContextGetter> url_request_context_getter);
+      scoped_refptr<net::URLRequestContextGetter> url_request_context_getter,
+      bool auto_refresh);

[Sergey Ulanov, 2014/05/19 22:33:46]

Why do we need both this flag and EnableAutoRefresh()?
I would prefer having only EnableAutoRefresh().

[rmsousa, 2014/05/19 22:46:53]

I forgot to delete EnableAutoRefresh() from here (there's no implementation).

I started with EnableAutoRefresh(), but started getting complicated (It'd either
have to be restricted so it's a one-shot that must be called just after the
constructor, before any CallWithToken calls, or we'd have to handle the cases
where AutoRefresh is enabled/disabled with a current token/expiration, or there
is a refresh in-flight, etc).

Let me know if you prefer either of those alternatives.

   virtual ~OAuthTokenGetter();
 
+  void EnableAutoRefresh(bool enabled);
+
   // Call |on_access_token| with an access token, or the failure status.
   void CallWithToken(const OAuthTokenGetter::TokenCallback& on_access_token);
 
   // gaia::GaiaOAuthClient::Delegate interface.
   virtual void OnGetTokensResponse(const std::string& user_email,
                                    const std::string& access_token,
                                    int expires_seconds) OVERRIDE;
   virtual void OnRefreshTokenResponse(const std::string& access_token,
                                       int expires_in_seconds) OVERRIDE;
   virtual void OnGetUserEmailResponse(const std::string& user_email) OVERRIDE;
   virtual void OnOAuthError() OVERRIDE;
   virtual void OnNetworkError(int response_code) OVERRIDE;
 
  private:
   void NotifyCallbacks(Status status,
                        const std::string& user_email,
                        const std::string& access_token);
   void RefreshOAuthToken();
 
   scoped_ptr<OAuthCredentials> oauth_credentials_;
   scoped_ptr<gaia::GaiaOAuthClient> gaia_oauth_client_;
   scoped_refptr<net::URLRequestContextGetter> url_request_context_getter_;
 
   bool refreshing_oauth_token_;
   std::string oauth_access_token_;
+  std::string verified_email_;
   base::Time auth_token_expiry_time_;
   std::queue<OAuthTokenGetter::TokenCallback> pending_callbacks_;
+  scoped_ptr<base::OneShotTimer<OAuthTokenGetter> > refresh_timer_;
 
   DISALLOW_COPY_AND_ASSIGN(OAuthTokenGetter);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_OAUTH_TOKEN_GETTER_H_