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

chrome/browser/profiles/avatar_menu_model_unittest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/profiles/avatar_menu_model.h"
 
 #include "base/string16.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/profiles/avatar_menu_model_observer.h"
 #include "chrome/browser/profiles/profile_info_cache.h"
@@ skipping 54 matching lines @@
 
   const AvatarMenuModel::Item& item1 = model.GetItemAt(0);
   EXPECT_EQ(0U, item1.model_index);
   EXPECT_EQ(name1, item1.name);
 
   const AvatarMenuModel::Item& item2 = model.GetItemAt(1);
   EXPECT_EQ(1U, item2.model_index);
   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"));
   string16 name2(ASCIIToUTF16("Test 2"));
 
   manager()->CreateTestingProfile("p1", name1, 0);
   manager()->CreateTestingProfile("p2", name2, 0);
 
   MockObserver observer;
   EXPECT_EQ(0, observer.change_count());
@@ skipping 17 matching lines @@
   const AvatarMenuModel::Item& item2 = model.GetItemAt(1);
   EXPECT_EQ(1U, item2.model_index);
   EXPECT_EQ(name2, item2.name);
 
   const AvatarMenuModel::Item& item3 = model.GetItemAt(2);
   EXPECT_EQ(2U, item3.model_index);
   EXPECT_EQ(name3, item3.name);
 }
 
 }  // namespace