[Mac/multiprofile] "Customize User..." should edit the currently active profile

chrome/browser/profiles/avatar_menu_model_unittest.cc

Index: chrome/browser/profiles/avatar_menu_model_unittest.cc
diff --git a/chrome/browser/profiles/avatar_menu_model_unittest.cc b/chrome/browser/profiles/avatar_menu_model_unittest.cc
index 3d3ca9eefb404647285fda8f711dde8203b445d1..978a58ffd8e887d56157376ed8c5a06665bb78a0 100644
--- a/chrome/browser/profiles/avatar_menu_model_unittest.cc
+++ b/chrome/browser/profiles/avatar_menu_model_unittest.cc
@@ -72,6 +72,18 @@ TEST_F(AvatarMenuModelTest, InitialCreation) {
   EXPECT_EQ(name2, item2.name);
 }
 
+TEST_F(AvatarMenuModelTest, ActiveItem) {
+  string16 name1(ASCIIToUTF16("Test 1"));
+  string16 name2(ASCIIToUTF16("Test 2"));
+
+  manager()->CreateTestingProfile("p1", name1, 0);
+  manager()->CreateTestingProfile("p2", name2, 0);
+
+  MockObserver observer;
+  AvatarMenuModel model(manager()->profile_info_cache(), &observer, browser());
+  ASSERT_EQ(2U, model.GetNumberOfItems());
+  ASSERT_EQ(0U, model.GetActiveProfileIndex());

[Robert Sesek, 2011/10/11 14:23:13]

May also want to change the active profile and assert that the new index is
correct. I think the profile_menu_controller_unittest does something like this,
too.

[jeremy, 2011/10/19 13:15:24]

You're right but I'm a bit pressed for time with this change, added TODO.

+}
 
 TEST_F(AvatarMenuModelTest, ChangeOnNotify) {
   string16 name1(ASCIIToUTF16("Test 1"));