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

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

Per mock:
https://beta-dot-folio.googleplex.com/chrome-ux/mocks/212-stars-iphone/20/phone#%2F05-detail-edit-bookmark.png%3Fc=hidden

All tags/filters associated with a bookmark should be displayed in
enhanced bookmark detail page. This CL introduced related APIs to jni
bridge for UI to retrieve.

BUG=NONE

Committed: https://crrev.com/a52ec9a58e74ea25bd071bd6a48ae64ba28b86cd
Cr-Commit-Position: refs/heads/master@{#302365}

[Ian Wen, 2014-10-31 23:10:54]

ianwen@chromium.org changed reviewers:
+ kkimlabs@chromium.org, tedchoc@chromium.org

[Ted C, 2014-10-31 23:18:22]

https://codereview.chromium.org/694093002/diff/1/chrome/browser/android/enhan...
File chrome/browser/android/enhanced_bookmarks/enhanced_bookmarks_bridge.cc
(right):

https://codereview.chromium.org/694093002/diff/1/chrome/browser/android/enhan...
chrome/browser/android/enhanced_bookmarks/enhanced_bookmarks_bridge.cc:97: jint
type) {
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]

https://codereview.chromium.org/694093002/diff/1/chrome/browser/android/enhan...
File chrome/browser/android/enhanced_bookmarks/enhanced_bookmarks_bridge.cc
(right):

https://codereview.chromium.org/694093002/diff/1/chrome/browser/android/enhan...
chrome/browser/android/enhanced_bookmarks/enhanced_bookmarks_bridge.cc:97: jint
type) {
On 2014/10/31 23:18:21, Ted C wrote:
> 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.

Done.

[Ted C, 2014-10-31 23:52:59]

lgtm

[Ian Wen, 2014-10-31 23:54:29]

The CQ bit was checked by ianwen@chromium.org

[commit-bot: I haz the power, 2014-10-31 23:55:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/694093002/40001

[commit-bot: I haz the power, 2014-11-01 00:38:31]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2014-11-01 00:39:29]

Patchset 3 (id:??) landed as
https://crrev.com/a52ec9a58e74ea25bd071bd6a48ae64ba28b86cd
Cr-Commit-Position: refs/heads/master@{#302365}