Index: chrome/browser/chromeos/profiles/profile_list_chromeos_unittest.cc |
diff --git a/chrome/browser/chromeos/profiles/profile_list_chromeos_unittest.cc b/chrome/browser/chromeos/profiles/profile_list_chromeos_unittest.cc |
index 09f7c1848f85bbdaf2651fe9b6d1403ddb1dc4bd..2d37fa451e0cc7c1d7949c9dd534dea4ec655874 100644 |
--- a/chrome/browser/chromeos/profiles/profile_list_chromeos_unittest.cc |
+++ b/chrome/browser/chromeos/profiles/profile_list_chromeos_unittest.cc |
@@ -227,7 +227,7 @@ TEST_F(ProfileListChromeOSTest, ModifyingNameResortsCorrectly) { |
AddProfile(name2, true); |
AvatarMenu* menu = GetAvatarMenu(); |
- |
+ EXPECT_EQ(0, change_count()); |
ASSERT_EQ(2U, menu->GetNumberOfItems()); |
const AvatarMenu::Item& item1 = menu->GetItemAt(0); |
@@ -243,9 +243,9 @@ TEST_F(ProfileListChromeOSTest, ModifyingNameResortsCorrectly) { |
GetFakeChromeUserManager()->SaveUserDisplayName( |
base::UTF16ToASCII(name1) + "@example.com", newname1); |
manager()->profile_info_cache()->SetNameOfProfileAtIndex(0, newname1); |
+ EXPECT_EQ(1, change_count()); |
const AvatarMenu::Item& item1next = menu->GetItemAt(0); |
- EXPECT_GT(change_count(), 1); |
EXPECT_EQ(0U, item1next.menu_index); |
EXPECT_EQ(name2, item1next.name); |
@@ -263,15 +263,15 @@ TEST_F(ProfileListChromeOSTest, ChangeOnNotify) { |
AvatarMenu* menu = GetAvatarMenu(); |
EXPECT_EQ(2U, menu->GetNumberOfItems()); |
+ EXPECT_EQ(0, change_count()); |
base::string16 name3(ASCIIToUTF16("p3.com")); |
AddProfile(name3, true); |
- // 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. |
- // TODO(michaelpg): Determine why actual change number does not match comment. |
- EXPECT_GE(change_count(), 4); |
+ // Three changes happened via the call to CreateTestingProfile: adding the |
+ // profile to the cache, setting the user name (which rebuilt the list of |
+ // profiles after the name change), and changing the avatar. |
+ EXPECT_EQ(change_count(), 3); |
ASSERT_EQ(3U, menu->GetNumberOfItems()); |
const AvatarMenu::Item& item1 = menu->GetItemAt(0); |