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

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

Issue 506353003: [Enhanced Bookmark] Add GetAllFoldersWithDepth in bookmark bridge (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add comments Created 6 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: chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java b/chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java
index 15a18f5ee2bbc377643ff60ec0eacb5a9ceb4e2c..17607dd961499eb12e2429126f876bcda4c35462 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java
@@ -215,6 +215,30 @@ public class BookmarksBridge {
}
/**
+ * Populates folderList with BookmarkIds of folders users can move bookmarks
+ * to and all folders have corresponding depth value in depthList. Folders
+ * having depths of 0 will be shown as top-layered folders. These include
+ * "Desktop Folder" itself as well as all children of "mobile" and "other".
+ * Children of 0-depth folders have depth of 1, and so on.
+ *
+ * The result list will be sorted alphabetically by title. "mobile", "other",
+ * root node, managed folder, partner folder are NOT included as results.
+ */
+ public void getAllFoldersWithDepths(List<BookmarkId> folderList,
+ List<Integer> depthList) {
+ assert mIsNativeBookmarkModelLoaded;
+ nativeGetAllFoldersWithDepths(mNativeBookmarksBridge, folderList, depthList);
+ }
+
+ /**
+ * @return The BookmarkId for Mobile folder node
+ */
+ public BookmarkId getMobileFolderId() {
+ assert mIsNativeBookmarkModelLoaded;
+ return nativeGetMobileFolderId(mNativeBookmarksBridge);
+ }
+
+ /**
* Reads sub-folder IDs, sub-bookmark IDs, or both of the given folder.
*
* @param getFolders Whether sub-folders should be returned.
@@ -445,6 +469,13 @@ public class BookmarksBridge {
}
@CalledByNative
+ private static void addToBookmarkIdListWithDepth(List<BookmarkId> folderList, long id,
+ int type, List<Integer> depthList, int depth) {
+ folderList.add(new BookmarkId(id, type));
+ depthList.add(depth);
+ }
+
+ @CalledByNative
private static BookmarkId createBookmarkId(long id, int type) {
return new BookmarkId(id, type);
}
@@ -459,6 +490,9 @@ public class BookmarksBridge {
boolean getNormal, List<BookmarkId> bookmarksList);
private native void nativeGetUncategorizedBookmarkIDs(long nativeBookmarksBridge,
List<BookmarkId> bookmarksList);
+ private native void nativeGetAllFoldersWithDepths(long nativeBookmarksBridge,
+ List<BookmarkId> folderList, List<Integer> depthList);
+ private native BookmarkId nativeGetMobileFolderId(long nativeBookmarksBridge);
private native void nativeGetChildIDs(long nativeBookmarksBridge, long id, int type,
boolean getFolders, boolean getBookmarks, List<BookmarkId> bookmarksList);
private native void nativeGetAllBookmarkIDsOrderedByCreationDate(long nativeBookmarksBridge,

Powered by Google App Engine
This is Rietveld 408576698