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

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

Issue 2610143002: Add RecentTabsPageTest (Closed)
Patch Set: Undo the prefs changes. Created 3 years, 11 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/RecentTabsRowAdapter.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsRowAdapter.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsRowAdapter.java
index e8e6920192e1c407a8cef2077889d8988fe945bd..3dedb2d2adf9665c4f16ad02f9524db5d787d650 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsRowAdapter.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsRowAdapter.java
@@ -28,13 +28,13 @@ import org.chromium.chrome.browser.favicon.FaviconHelper.FaviconImageCallback;
import org.chromium.chrome.browser.ntp.ForeignSessionHelper.ForeignSession;
import org.chromium.chrome.browser.ntp.ForeignSessionHelper.ForeignSessionTab;
import org.chromium.chrome.browser.ntp.ForeignSessionHelper.ForeignSessionWindow;
-import org.chromium.chrome.browser.ntp.RecentlyClosedBridge.RecentlyClosedTab;
import org.chromium.chrome.browser.signin.SigninAccessPoint;
import org.chromium.chrome.browser.signin.SigninAndSyncView;
import org.chromium.ui.base.DeviceFormFactor;
import org.chromium.ui.mojom.WindowOpenDisposition;
import java.util.ArrayList;
+import java.util.List;
/**
* Row adapter for presenting recently closed tabs, synced tabs from other devices, the sync or
@@ -65,7 +65,7 @@ public class RecentTabsRowAdapter extends BaseExpandableListAdapter {
}
private final Activity mActivity;
- private final ArrayList<Group> mGroups;
+ private final List<Group> mGroups;
private final Drawable mDefaultFavicon;
private final RecentTabsManager mRecentTabsManager;
private final RecentlyClosedTabsGroup mRecentlyClosedTabsGroup = new RecentlyClosedTabsGroup();
@@ -349,8 +349,8 @@ public class RecentTabsRowAdapter extends BaseExpandableListAdapter {
* page.
*/
class RecentlyClosedTabsGroup extends Group {
- private static final int ID_OPEN_IN_NEW_TAB = 1;
- private static final int ID_REMOVE_ALL = 2;
+ static final int ID_OPEN_IN_NEW_TAB = 1;
+ static final int ID_REMOVE_ALL = 2;
@Override
public GroupType getGroupType() {
@@ -629,7 +629,7 @@ public class RecentTabsRowAdapter extends BaseExpandableListAdapter {
public RecentTabsRowAdapter(Activity activity, RecentTabsManager recentTabsManager) {
mActivity = activity;
mRecentTabsManager = recentTabsManager;
- mGroups = new ArrayList<Group>();
+ mGroups = new ArrayList<>();
mFaviconCache = buildFaviconCache(MAX_NUM_FAVICONS_TO_CACHE);
Resources resources = activity.getResources();

Powered by Google App Engine
This is Rietveld 408576698