GitLab

When migrating bookmarks sync to USS architecture, it assumes that
the bookmark model is creating only once.
ManagedBookmarkServiceTest.EmptyManagedNode is creating the model
twice which doesn't match the expecations of the new architecture.

This CL refactor the test to make sure the bookmark model is created
only once. In fact, the new scenario being tested resembles
reality a lot closer, since keyed services are never destroyed during
the lifetime of a profile.

Bug: 516866
Change-Id: I4a966270cc6bfcbdb7e1c87a096fea6a81e4b1b4
Reviewed-on: https://chromium-review.googlesource.com/1243075Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org>
Cr-Commit-Position: refs/heads/master@{#594496}