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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java

Issue 1901623002: Unified Bookmarks and Recent Tabs signin methods. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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: chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java
index c7993199dee60dcd293b80ec3c30f7d6a134ab20..9ad2484a097eb1418a655019c867a4caf27a74c9 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java
@@ -9,7 +9,6 @@ import android.content.SharedPreferences;
import android.graphics.Bitmap;
import android.preference.PreferenceManager;
-import org.chromium.base.ObserverList;
import org.chromium.base.ThreadUtils;
import org.chromium.chrome.browser.UrlConstants;
import org.chromium.chrome.browser.favicon.FaviconHelper;
@@ -20,14 +19,11 @@ import org.chromium.chrome.browser.metrics.StartupMetrics;
import org.chromium.chrome.browser.ntp.ForeignSessionHelper.ForeignSession;
import org.chromium.chrome.browser.ntp.ForeignSessionHelper.ForeignSessionCallback;
import org.chromium.chrome.browser.ntp.ForeignSessionHelper.ForeignSessionTab;
-import org.chromium.chrome.browser.ntp.RecentTabsPromoView.SyncPromoModel;
import org.chromium.chrome.browser.ntp.RecentlyClosedBridge.RecentlyClosedCallback;
import org.chromium.chrome.browser.ntp.RecentlyClosedBridge.RecentlyClosedTab;
import org.chromium.chrome.browser.profiles.Profile;
-import org.chromium.chrome.browser.signin.SigninAccessPoint;
import org.chromium.chrome.browser.signin.SigninManager;
import org.chromium.chrome.browser.signin.SigninManager.SignInStateObserver;
-import org.chromium.chrome.browser.sync.ProfileSyncService;
import org.chromium.chrome.browser.tab.Tab;
import org.chromium.content_public.browser.LoadUrlParams;
import org.chromium.sync.AndroidSyncSettings;
@@ -40,8 +36,7 @@ import java.util.List;
/**
* Provides the domain logic and data for RecentTabsPage and RecentTabsRowAdapter.
*/
-public class RecentTabsManager implements AndroidSyncSettingsObserver, SignInStateObserver,
- SyncPromoModel {
+public class RecentTabsManager implements AndroidSyncSettingsObserver, SignInStateObserver {
/**
* Implement this to receive updates when the page contents change.
@@ -60,8 +55,6 @@ public class RecentTabsManager implements AndroidSyncSettingsObserver, SignInSta
private final Profile mProfile;
private final Tab mTab;
private final Context mContext;
- private final ObserverList<AndroidSyncSettingsObserver> mObservers =
- new ObserverList<AndroidSyncSettingsObserver>();
private FaviconHelper mFaviconHelper;
private ForeignSessionHelper mForeignSessionHelper;
@@ -461,52 +454,11 @@ public class RecentTabsManager implements AndroidSyncSettingsObserver, SignInSta
if (mIsDestroyed) return;
updateForeignSessions();
postUpdate();
- for (AndroidSyncSettingsObserver observer : mObservers) {
- observer.androidSyncSettingsChanged();
- }
}
});
}
- // SyncPromoModel
- @Override
- public boolean isSyncEnabled() {
- return AndroidSyncSettings.isSyncEnabled(mContext);
- }
-
- @Override
public boolean isSignedIn() {
return ChromeSigninController.get(mContext).isSignedIn();
}
-
- @Override
- public void enableSync() {
- ProfileSyncService syncService = ProfileSyncService.get();
- if (syncService != null) {
- syncService.requestStart();
- }
- }
-
- @Override
- public void registerForSyncUpdates(AndroidSyncSettingsObserver changeListener) {
- mObservers.addObserver(changeListener);
- }
-
- @Override
- public void unregisterForSyncUpdates(AndroidSyncSettingsObserver changeListener) {
- mObservers.removeObserver(changeListener);
- }
-
- @Override
- public ProfileDataCache getProfileDataCache() {
- if (mProfileDataCache == null) {
- mProfileDataCache = new ProfileDataCache(mContext, Profile.getLastUsedProfile());
- }
- return mProfileDataCache;
- }
-
- @Override
- public int getAccessPoint() {
- return SigninAccessPoint.RECENT_TABS;
- }
}

Powered by Google App Engine
This is Rietveld 408576698