GitLab

This patch refactors the SyncedBookmarkTracker API to make it less
sync-ID-centric (aka server IDs). Instead, entity pointers are proposed
(const SyncedBookmarkTracker::Entity*) as handles, following the design
principle in BookmarkModel and const BookmarkNode*.

This spares a few lookups but otherwise introduces no behavioral
changes.

Change-Id: Idfcaff7dfc268a29ca3f52e68b54ae11a51c6c71
Bug: 1032052
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2094982Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Mohamed Amir Yosef <mamir@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748675}