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

Unified Diff: components/signin/core/browser/android/java/src/org/chromium/components/signin/ChromeSigninController.java

Issue 2351703003: Refactor: move components/sync/../signin to components/signin/../ (Closed)
Patch Set: rebase Created 4 years, 3 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
Index: components/signin/core/browser/android/java/src/org/chromium/components/signin/ChromeSigninController.java
diff --git a/components/sync/android/java/src/org/chromium/components/sync/signin/ChromeSigninController.java b/components/signin/core/browser/android/java/src/org/chromium/components/signin/ChromeSigninController.java
similarity index 87%
rename from components/sync/android/java/src/org/chromium/components/sync/signin/ChromeSigninController.java
rename to components/signin/core/browser/android/java/src/org/chromium/components/signin/ChromeSigninController.java
index 09702230b6c6af1798584f89332eb02b49bed08f..38c136c27119f3e083d3e50992f2cb038b72278a 100644
--- a/components/sync/android/java/src/org/chromium/components/sync/signin/ChromeSigninController.java
+++ b/components/signin/core/browser/android/java/src/org/chromium/components/signin/ChromeSigninController.java
@@ -2,13 +2,12 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-package org.chromium.components.sync.signin;
+package org.chromium.components.signin;
import android.accounts.Account;
import android.content.Context;
import org.chromium.base.ContextUtils;
-import org.chromium.components.sync.AndroidSyncSettings;
/**
* Caches the signed-in username in the app prefs.
@@ -27,7 +26,6 @@ public class ChromeSigninController {
private ChromeSigninController(Context context) {
mApplicationContext = context.getApplicationContext();
- AndroidSyncSettings.updateAccount(context, getSignedInUser());
}
/**
@@ -62,8 +60,6 @@ public class ChromeSigninController {
.edit()
.putString(SIGNED_IN_ACCOUNT_KEY, accountName)
.apply();
- // TODO(maxbogue): Move this to SigninManager.
- AndroidSyncSettings.updateAccount(mApplicationContext, getSignedInUser());
}
public String getSignedInAccountName() {

Powered by Google App Engine
This is Rietveld 408576698