GitLab

BookmarkItemsAdapter currently uses the backend data class BookmarkItem
as its data object. This is not flexible to add decorative or non
bookmark ViewHolders to its recycler view.

Also the sync promo header uses null to identify itself to the recycler
view, which is kind of hacky, and UI logic assumes null is the place
holder for sync promo header.

This CL introduces BookmarkListEntry, which can represent any kind of
ViewHolders, and carries different kind of data objects. Sync promo
header also has a BookmarkListEntry object.

Bug: 1128074
Change-Id: Id11c0164eb06cf3a59b35eb6cad7e33e4f9d19f4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461724
Commit-Queue: Xing Liu <xingliu@chromium.org>
Reviewed-by: Brandon Wylie <wylieb@chromium.org>
Reviewed-by: Theresa  <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#816421}