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

Unified Diff: components/signin/core/browser/account_reconcilor.cc

Issue 297303003: Revert of non-new-profile-management creates a "no-op" style account_reconcilor, (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/signin/core/browser/account_reconcilor.h ('k') | google_apis/gaia/fake_gaia.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/signin/core/browser/account_reconcilor.cc
diff --git a/components/signin/core/browser/account_reconcilor.cc b/components/signin/core/browser/account_reconcilor.cc
index 8684d9b5791885e4cf0c702eee23524c92f97e18..4d7cf9789b867f93ecd230d2f2f755203ae88023 100644
--- a/components/signin/core/browser/account_reconcilor.cc
+++ b/components/signin/core/browser/account_reconcilor.cc
@@ -17,7 +17,6 @@
#include "components/signin/core/browser/signin_client.h"
#include "components/signin/core/browser/signin_metrics.h"
#include "components/signin/core/browser/signin_oauth_helper.h"
-#include "components/signin/core/common/profile_management_switches.h"
#include "google_apis/gaia/gaia_auth_fetcher.h"
#include "google_apis/gaia/gaia_auth_util.h"
#include "google_apis/gaia/gaia_constants.h"
@@ -335,7 +334,7 @@
void AccountReconcilor::OnRefreshTokenRevoked(const std::string& account_id) {
VLOG(1) << "AccountReconcilor::OnRefreshTokenRevoked: " << account_id;
- PerformStartRemoveAction(account_id);
+ StartRemoveAction(account_id);
}
void AccountReconcilor::OnRefreshTokensLoaded() {}
@@ -357,28 +356,22 @@
}
void AccountReconcilor::PerformMergeAction(const std::string& account_id) {
- if (!switches::IsNewProfileManagement())
- return;
VLOG(1) << "AccountReconcilor::PerformMergeAction: " << account_id;
merge_session_helper_.LogIn(account_id);
}
-void AccountReconcilor::PerformStartRemoveAction(
- const std::string& account_id) {
- VLOG(1) << "AccountReconcilor::PerformStartRemoveAction: " << account_id;
- GetAccountsFromCookie(base::Bind(
- &AccountReconcilor::PerformFinishRemoveAction,
- base::Unretained(this),
- account_id));
-}
-
-void AccountReconcilor::PerformFinishRemoveAction(
+void AccountReconcilor::StartRemoveAction(const std::string& account_id) {
+ VLOG(1) << "AccountReconcilor::StartRemoveAction: " << account_id;
+ GetAccountsFromCookie(base::Bind(&AccountReconcilor::FinishRemoveAction,
+ base::Unretained(this),
+ account_id));
+}
+
+void AccountReconcilor::FinishRemoveAction(
const std::string& account_id,
const GoogleServiceAuthError& error,
const std::vector<std::pair<std::string, bool> >& accounts) {
- if (!switches::IsNewProfileManagement())
- return;
- VLOG(1) << "AccountReconcilor::PerformFinishRemoveAction:"
+ VLOG(1) << "AccountReconcilor::FinishRemoveAction:"
<< " account=" << account_id << " error=" << error.ToString();
if (error.state() == GoogleServiceAuthError::NONE) {
AbortReconcile();
@@ -396,8 +389,6 @@
void AccountReconcilor::PerformAddToChromeAction(const std::string& account_id,
int session_index) {
- if (!switches::IsNewProfileManagement())
- return;
VLOG(1) << "AccountReconcilor::PerformAddToChromeAction:"
<< " account=" << account_id << " session_index=" << session_index;
@@ -408,8 +399,6 @@
}
void AccountReconcilor::PerformLogoutAllAccountsAction() {
- if (!switches::IsNewProfileManagement())
- return;
VLOG(1) << "AccountReconcilor::PerformLogoutAllAccountsAction";
merge_session_helper_.LogOutAllAccounts();
}
@@ -723,22 +712,13 @@
FinishReconcile();
}
-void AccountReconcilor::PerformAddAccountToTokenService(
- const std::string& account_id,
- const std::string& refresh_token) {
- // The flow should never get to this method if new_profile_management is
- // false, but better safe than sorry.
- if (!switches::IsNewProfileManagement())
- return;
- token_service_->UpdateCredentials(account_id, refresh_token);
-}
-
void AccountReconcilor::HandleRefreshTokenFetched(
const std::string& account_id,
const std::string& refresh_token) {
if (!refresh_token.empty()) {
- PerformAddAccountToTokenService(account_id, refresh_token);
- }
+ token_service_->UpdateCredentials(account_id, refresh_token);
+ }
+
// Remove the account from the list that is being updated.
for (std::vector<std::pair<std::string, int> >::iterator i =
add_to_chrome_.begin();
« no previous file with comments | « components/signin/core/browser/account_reconcilor.h ('k') | google_apis/gaia/fake_gaia.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698