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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkDrawerListViewAdapter.java

Issue 1935413003: [Android Bookmark] Remove all bookmark section from bookmark manager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: make search available in every folder 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/bookmarks/BookmarkDrawerListViewAdapter.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkDrawerListViewAdapter.java b/chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkDrawerListViewAdapter.java
index 995c5460666efe8bae25f930e6ddbc0986788714..9215463cb1c982545e76698728a28ef2aea31b25 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkDrawerListViewAdapter.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkDrawerListViewAdapter.java
@@ -23,9 +23,8 @@ import java.util.Set;
*/
class BookmarkDrawerListViewAdapter extends BaseAdapter {
static final int TYPE_FOLDER = 0;
- static final int TYPE_ALL_ITEMS = -1;
- static final int TYPE_DIVIDER = -2;
- static final int TYPE_FOLDERS_TITLE = -3;
+ static final int TYPE_DIVIDER = -1;
+ static final int TYPE_FOLDERS_TITLE = -2;
static final int VIEW_TYPE_ITEM = 0;
static final int VIEW_TYPE_DIVIDER = 1;
@@ -34,9 +33,8 @@ class BookmarkDrawerListViewAdapter extends BaseAdapter {
private BookmarkDelegate mDelegate;
private List<Item> mTopSection = new ArrayList<Item>();
private List<Item> mMiddleSection = new ArrayList<Item>();
- private List<Item> mBottomSection = new ArrayList<Item>();
// array containing the order of sections
- private List<?>[] mSections = {mTopSection, mMiddleSection, mBottomSection};
+ private List<?>[] mSections = {mTopSection, mMiddleSection};
gone 2016/05/04 18:21:52 Kind of weird to have a top and middle section but
Ian Wen 2016/05/04 20:21:55 Good point! Done.
private BookmarkId mDesktopNodeId = null;
private BookmarkId mMobileNodeId = null;
@@ -91,8 +89,6 @@ class BookmarkDrawerListViewAdapter extends BaseAdapter {
private void repopulateTopSection() {
mTopSection.clear();
- mTopSection.add(new Item(TYPE_ALL_ITEMS));
-
if (mDelegate.getModel().isFolderVisible(mMobileNodeId)) {
mTopSection.add(new Item(mMobileNodeId));
}
@@ -102,7 +98,6 @@ class BookmarkDrawerListViewAdapter extends BaseAdapter {
if (mDelegate.getModel().isFolderVisible(mOthersNodeId)) {
mTopSection.add(new Item(mOthersNodeId));
}
-
if (mManagedAndPartnerFolderIds != null) {
for (BookmarkId id : mManagedAndPartnerFolderIds) {
mTopSection.add(new Item(id));
@@ -145,7 +140,6 @@ class BookmarkDrawerListViewAdapter extends BaseAdapter {
void clear() {
mTopSection.clear();
mMiddleSection.clear();
- mBottomSection.clear();
}
void setBookmarkUIDelegate(BookmarkDelegate delegate) {
@@ -184,9 +178,7 @@ class BookmarkDrawerListViewAdapter extends BaseAdapter {
* Get item position of the given mode.
*/
int getItemPosition(int state, Object modeDetail) {
- if (state == BookmarkUIState.STATE_ALL_BOOKMARKS) {
- return 0;
- } else if (state == BookmarkUIState.STATE_FOLDER) {
+ if (state == BookmarkUIState.STATE_FOLDER) {
Set<BookmarkId> topLevelFolderParents = new HashSet<>();
topLevelFolderParents.addAll(mDelegate.getModel().getTopLevelFolderParentIDs());
topLevelFolderParents.add(mDesktopNodeId);
@@ -303,11 +295,6 @@ class BookmarkDrawerListViewAdapter extends BaseAdapter {
int iconDrawableId;
switch (item.mType) {
- case TYPE_ALL_ITEMS:
- title = listItemView.getContext().getResources().getString(
- R.string.bookmark_drawer_all_items);
- iconDrawableId = R.drawable.btn_star;
- break;
case TYPE_FOLDER:
title = mDelegate.getModel().getBookmarkById(item.mFolderId).getTitle();
if (mManagedAndPartnerFolderIds != null

Powered by Google App Engine
This is Rietveld 408576698