Index: chrome/browser/profiles/profile_list_desktop_unittest.cc |
diff --git a/chrome/browser/profiles/profile_list_desktop_unittest.cc b/chrome/browser/profiles/profile_list_desktop_unittest.cc |
index 95f03c4dfc3e29088b28f6266b47206a568429aa..51905c88a2d92d1dab3ff124d98680db71cb498f 100644 |
--- a/chrome/browser/profiles/profile_list_desktop_unittest.cc |
+++ b/chrome/browser/profiles/profile_list_desktop_unittest.cc |
@@ -226,7 +226,7 @@ TEST_F(ProfileListDesktopTest, ModifyingNameResortsCorrectly) { |
manager()->profile_info_cache()->SetNameOfProfileAtIndex(0, |
ASCIIToUTF16(newname1)); |
const AvatarMenu::Item& item1next = model->GetItemAt(0); |
- EXPECT_GT(change_count(), 1); |
+ EXPECT_EQ(1, change_count()); |
EXPECT_EQ(0U, item1next.menu_index); |
EXPECT_EQ(ASCIIToUTF16(name2), item1next.name); |
@@ -245,14 +245,12 @@ TEST_F(ProfileListDesktopTest, ChangeOnNotify) { |
manager()->CreateTestingProfile("Test 3"); |
- // Four changes happened via the call to CreateTestingProfile: adding the |
- // profile to the cache, setting the user name, rebuilding the list of |
- // profiles after the name change, and changing the avatar. |
+ // Three changes happened via the call to CreateTestingProfile: adding the |
+ // profile to the cache, setting the user name (which rebuilds the list of |
+ // profiles after the name change) and changing the avatar. |
// On Windows, an extra change happens to set the shortcut name for the |
// profile. |
- // TODO(michaelpg): Determine why five changes happen on ChromeOS and |
- // investigate other platforms. |
- EXPECT_GE(change_count(), 4); |
+ EXPECT_GE(3, change_count()); |
ASSERT_EQ(3U, model->GetNumberOfItems()); |
const AvatarMenu::Item& item1 = model->GetItemAt(0); |