Add method to get filters for a bookmark in Enhanced bookmark bridge

chrome/browser/android/enhanced_bookmarks/enhanced_bookmarks_bridge.cc

Index: chrome/browser/android/enhanced_bookmarks/enhanced_bookmarks_bridge.cc
diff --git a/chrome/browser/android/enhanced_bookmarks/enhanced_bookmarks_bridge.cc b/chrome/browser/android/enhanced_bookmarks/enhanced_bookmarks_bridge.cc
index ed4016aa764a1910a4c61f4883f1da126a30c0bb..5f1a3cbb34822d0dfb3d0536b127915a3b417429 100644
--- a/chrome/browser/android/enhanced_bookmarks/enhanced_bookmarks_bridge.cc
+++ b/chrome/browser/android/enhanced_bookmarks/enhanced_bookmarks_bridge.cc
@@ -90,6 +90,19 @@ void EnhancedBookmarksBridge::SetBookmarkDescription(JNIEnv* env,
       node, base::android::ConvertJavaStringToUTF8(env, description));
 }
 
+ScopedJavaLocalRef<jobjectArray> EnhancedBookmarksBridge::GetFiltersForBookmark(
+    JNIEnv* env,
+    jobject obj,
+    jlong id,
+    jint type) {

[Ted C, 2014/10/31 23:18:21]

type isn't used, but it probably should be.

If it's a BOOKMARK_TYPE_PARTNER then you either should be returning null or an
empty filter list.  Or you should be CHECKing that the type passed in is
TYPE_NORMAL because that is the only one where filters are applicable.

That and the id space for partner bookmark and regular can overlap so this could
also just return the wrong information if called on a partner bookmark.

[Ian Wen, 2014/10/31 23:46:23]

Done.

+  DCHECK(enhanced_bookmark_model_->loaded());
+  const BookmarkNode* node = bookmarks::GetBookmarkNodeByID(
+        enhanced_bookmark_model_->bookmark_model(), static_cast<int64>(id));
+  std::vector<std::string> filters =
+      cluster_service_->ClustersForBookmark(node);
+  return base::android::ToJavaArrayOfStrings(env, filters);
+}
+
 void EnhancedBookmarksBridge::GetBookmarksForFilter(JNIEnv* env,
                                                     jobject obj,
                                                     jstring j_filter,