GitLab

BookmarkExpandedStateTrackerTest passes an empty path into
BookmarkModel::Load(), which causes the loader to look for a "Bookmarks"
file in the current directory. Asides from the attached issue, this
means the test behavior may change if the current directory does happen
to have a "Bookmarks" file.

This CL adds a base::ScopedTempDir to the test, making it not depend on
the current directory.

Bug: 1116809
Change-Id: I18db1ffdbdb2ae29aa17fef8920dde1ba8638f6a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2359389
Auto-Submit: Victor Costan <pwnall@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#798692}