Fixed flakiness in GetMostRecentlyAddedUserNodeForURLSkipsManagedNodes.

base::Time::Now() doesn't guarantee that successive calls don't get the
same value, and this seems to be happening on some Windows systems when
two BookmarkNodes are created one after the other in a unit test.

R=mad@chromium.org
TBR=sky@chromium.org
BUG=None

Review URL: https://codereview.chromium.org/348893002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278805 0039d316-1c4b-4281-b951-d872f2087c98
parent 48ba63f0
...@@ -1173,7 +1173,7 @@ TEST_F(BookmarkModelTest, GetMostRecentlyAddedUserNodeForURLSkipsManagedNodes) { ...@@ -1173,7 +1173,7 @@ TEST_F(BookmarkModelTest, GetMostRecentlyAddedUserNodeForURLSkipsManagedNodes) {
// Having a more recent managed node doesn't count either. // Having a more recent managed node doesn't count either.
const BookmarkNode* managed = model_->AddURL(managed_parent, 0, title, url); const BookmarkNode* managed = model_->AddURL(managed_parent, 0, title, url);
EXPECT_GT(managed->date_added(), user->date_added()); EXPECT_GE(managed->date_added(), user->date_added());
EXPECT_EQ(user, model_->GetMostRecentlyAddedUserNodeForURL(url)); EXPECT_EQ(user, model_->GetMostRecentlyAddedUserNodeForURL(url));
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment