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

Unified Diff: sync/android/java/src/org/chromium/sync/AndroidSyncSettings.java

Issue 1145003002: [Sync] Finish transition to new AndroidSyncSettings interface. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/android/java/src/org/chromium/sync/AndroidSyncSettings.java
diff --git a/sync/android/java/src/org/chromium/sync/AndroidSyncSettings.java b/sync/android/java/src/org/chromium/sync/AndroidSyncSettings.java
index c36cef484eef50328b7077832ac8f6051137f981..f43f290042e75e8b477f1bf78357d53eac3d51ea 100644
--- a/sync/android/java/src/org/chromium/sync/AndroidSyncSettings.java
+++ b/sync/android/java/src/org/chromium/sync/AndroidSyncSettings.java
@@ -291,67 +291,7 @@ public class AndroidSyncSettings {
}
}
- // TODO(maxbogue): make private once downstream uses are removed.
- @Deprecated
- public String getContractAuthority() {
+ private String getContractAuthority() {
return mApplicationContext.getPackageName();
}
-
- // Deprecated section; to be removed once downstream no longer uses them.
-
- @Deprecated
- public static AndroidSyncSettings get(Context context) {
- ensureInitialized(context);
- return sInstance;
- }
-
- @Deprecated
- public boolean isSyncEnabled() {
- return mMasterSyncEnabled && mChromeSyncEnabled;
- }
-
- @Deprecated
- public boolean isChromeSyncEnabled() {
- return mChromeSyncEnabled;
- }
-
- @Deprecated
- public boolean isMasterSyncEnabled() {
- return mMasterSyncEnabled;
- }
-
- @Deprecated
- public void enableChromeSync() {
- setChromeSyncEnabled(true);
- }
-
- @Deprecated
- public void disableChromeSync() {
- setChromeSyncEnabled(false);
- }
-
- @Deprecated
- public void updateAccount(Account account) {
- synchronized (mLock) {
- mAccount = account;
- updateSyncability();
- }
- if (updateCachedSettings()) {
- notifyObservers();
- }
- }
-
- @Deprecated
- public void registerObserver(AndroidSyncSettingsObserver observer) {
- synchronized (mLock) {
- mObservers.addObserver(observer);
- }
- }
-
- @Deprecated
- public void unregisterObserver(AndroidSyncSettingsObserver observer) {
- synchronized (mLock) {
- mObservers.removeObserver(observer);
- }
- }
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698