Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2017 The Chromium Authors. All rights reserved. | 1 // Copyright 2017 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 #ifndef CHROME_BROWSER_SIGNIN_FORCE_SIGNIN_VERIFIER_H_ | 5 #ifndef CHROME_BROWSER_SIGNIN_FORCE_SIGNIN_VERIFIER_H_ |
| 6 #define CHROME_BROWSER_SIGNIN_FORCE_SIGNIN_VERIFIER_H_ | 6 #define CHROME_BROWSER_SIGNIN_FORCE_SIGNIN_VERIFIER_H_ |
| 7 | 7 |
| 8 #include <memory> | 8 #include <memory> |
| 9 #include <string> | 9 #include <string> |
| 10 | 10 |
| 11 #include "base/macros.h" | 11 #include "base/macros.h" |
| 12 #include "base/time/time.h" | 12 #include "base/time/time.h" |
| 13 #include "base/timer/timer.h" | 13 #include "base/timer/timer.h" |
| 14 #include "google_apis/gaia/oauth2_token_service.h" | 14 #include "google_apis/gaia/oauth2_token_service.h" |
| 15 #include "net/base/backoff_entry.h" | 15 #include "net/base/backoff_entry.h" |
| 16 #include "net/base/network_change_notifier.h" | 16 #include "net/base/network_change_notifier.h" |
| 17 | 17 |
| 18 namespace views { | |
| 19 class Widget; | |
| 20 class WidgetDeletionObserver; | |
| 21 } // namespace views | |
| 22 | |
| 18 class Profile; | 23 class Profile; |
| 19 class SigninManager; | 24 class SigninManager; |
| 20 | 25 |
| 21 // ForceSigninVerifier will verify profile's auth token when profile is loaded | 26 // ForceSigninVerifier will verify profile's auth token when profile is loaded |
| 22 // into memory by the first time via gaia server. It will retry on any transient | 27 // into memory by the first time via gaia server. It will retry on any transient |
| 23 // error. | 28 // error. |
| 24 class ForceSigninVerifier | 29 class ForceSigninVerifier |
| 25 : public OAuth2TokenService::Consumer, | 30 : public OAuth2TokenService::Consumer, |
| 26 public net::NetworkChangeNotifier::NetworkChangeObserver { | 31 public net::NetworkChangeNotifier::NetworkChangeObserver { |
| 27 public: | 32 public: |
| 28 explicit ForceSigninVerifier(Profile* profile); | 33 explicit ForceSigninVerifier(Profile* profile); |
| 29 ~ForceSigninVerifier() override; | 34 ~ForceSigninVerifier() override; |
| 30 | 35 |
| 31 // OAuth2TokenService::Consumer implementation | 36 // override OAuth2TokenService::Consumer |
| 32 void OnGetTokenSuccess(const OAuth2TokenService::Request* request, | 37 void OnGetTokenSuccess(const OAuth2TokenService::Request* request, |
| 33 const std::string& access_token, | 38 const std::string& access_token, |
| 34 const base::Time& expiration_time) override; | 39 const base::Time& expiration_time) override; |
| 35 void OnGetTokenFailure(const OAuth2TokenService::Request* request, | 40 void OnGetTokenFailure(const OAuth2TokenService::Request* request, |
| 36 const GoogleServiceAuthError& error) override; | 41 const GoogleServiceAuthError& error) override; |
| 37 | 42 |
| 38 // net::NetworkChangeNotifier::NetworkChangeObserver | 43 // override net::NetworkChangeNotifier::NetworkChangeObserver |
| 39 void OnNetworkChanged( | 44 void OnNetworkChanged( |
| 40 net::NetworkChangeNotifier::ConnectionType type) override; | 45 net::NetworkChangeNotifier::ConnectionType type) override; |
| 41 | 46 |
| 42 // Cancel any pending or ongoing verification. | 47 // Cancel any pending or ongoing verification. |
| 43 void Cancel(); | 48 void Cancel(); |
| 44 | 49 |
| 45 // Return the value of |has_token_verified_|. | 50 // Return the value of |has_token_verified_|. |
| 46 bool HasTokenBeenVerified(); | 51 bool HasTokenBeenVerified(); |
| 47 | 52 |
| 48 protected: | 53 protected: |
| 49 // Send the token verification request. The request will be sent only if | 54 // Send the token verification request. The request will be sent only if |
| 50 // - The token has never been verified before. | 55 // - The token has never been verified before. |
| 51 // - There is no on going verification. | 56 // - There is no on going verification. |
| 52 // - There is network connection. | 57 // - There is network connection. |
| 53 // - The profile has signed in. | 58 // - The profile has signed in. |
| 54 // | 59 // |
| 55 void SendRequest(); | 60 void SendRequest(); |
| 56 | 61 |
| 57 virtual bool ShouldSendRequest(); | 62 virtual bool ShouldSendRequest(); |
| 58 | 63 |
| 59 // Show the warning dialog before signing out user and closing assoicated | 64 // Show the warning dialog before signing out user and closing assoicated |
| 60 // browser window. | 65 // browser window. |
| 61 virtual void ShowDialog(); | 66 virtual void ShowDialog(); |
| 62 | 67 |
| 68 // Start the window closing countdown, return the duration. | |
| 69 base::TimeDelta StartCountdown(); | |
| 70 | |
| 63 OAuth2TokenService::Request* GetRequestForTesting(); | 71 OAuth2TokenService::Request* GetRequestForTesting(); |
| 64 net::BackoffEntry* GetBackoffEntryForTesting(); | 72 net::BackoffEntry* GetBackoffEntryForTesting(); |
| 65 base::OneShotTimer* GetOneShotTimerForTesting(); | 73 base::OneShotTimer* GetOneShotTimerForTesting(); |
| 74 base::OneShotTimer* GetWindowCloseTimerForTesting(); | |
| 66 | 75 |
| 67 private: | 76 private: |
| 77 // Close all browser windows. | |
| 78 void CloseWindows(); | |
|
Roger Tawa OOO till Jul 10th
2017/06/20 14:17:52
Nit: rename function to CloseAllBrowserWindows() a
zmin
2017/06/20 17:40:32
Done.
| |
| 79 | |
| 68 std::unique_ptr<OAuth2TokenService::Request> access_token_request_; | 80 std::unique_ptr<OAuth2TokenService::Request> access_token_request_; |
| 69 | 81 |
| 82 #if !defined(OS_MACOSX) | |
| 83 Profile* profile_; | |
| 84 #endif | |
| 70 // Indicates whether the verification is finished successfully or with a | 85 // Indicates whether the verification is finished successfully or with a |
| 71 // persistent error. | 86 // persistent error. |
| 72 bool has_token_verified_; | 87 bool has_token_verified_; |
| 73 net::BackoffEntry backoff_entry_; | 88 net::BackoffEntry backoff_entry_; |
| 74 base::OneShotTimer backoff_request_timer_; | 89 base::OneShotTimer backoff_request_timer_; |
| 75 | 90 |
| 76 OAuth2TokenService* oauth2_token_service_; | 91 OAuth2TokenService* oauth2_token_service_; |
| 77 SigninManager* signin_manager_; | 92 SigninManager* signin_manager_; |
| 78 | 93 |
| 79 base::Time token_request_time_; | 94 base::Time token_request_time_; |
| 80 | 95 |
| 96 // Window closing timer. | |
| 97 base::OneShotTimer window_close_timer_; | |
| 98 // The widget of dialog. | |
|
Roger Tawa OOO till Jul 10th
2017/06/20 14:17:52
Comments here are not very descriptive. I'd remov
zmin
2017/06/20 17:40:32
Done.
| |
| 99 views::Widget* reauth_dialog_; | |
| 100 std::unique_ptr<views::WidgetDeletionObserver> dialog_observer_; | |
| 101 | |
| 81 DISALLOW_COPY_AND_ASSIGN(ForceSigninVerifier); | 102 DISALLOW_COPY_AND_ASSIGN(ForceSigninVerifier); |
| 82 }; | 103 }; |
| 83 | 104 |
| 84 #endif // CHROME_BROWSER_SIGNIN_FORCE_SIGNIN_VERIFIER_H_ | 105 #endif // CHROME_BROWSER_SIGNIN_FORCE_SIGNIN_VERIFIER_H_ |
| OLD | NEW |