Multi-Profiles: Add icon chooser to profiles menu

chrome/browser/ui/profile_menu_model.h

Index: chrome/browser/ui/profile_menu_model.h
diff --git a/chrome/browser/ui/profile_menu_model.h b/chrome/browser/ui/profile_menu_model.h
index 369430453e5d2663a74542e1ccadc9923367c88c..3e5034304c706e1c2740ba73b0145d82f48501f7 100644
--- a/chrome/browser/ui/profile_menu_model.h
+++ b/chrome/browser/ui/profile_menu_model.h
@@ -10,11 +10,7 @@
 
 class Browser;
 
-// ProfileMenuModel
-//
-// Menu for the multi-profile button displayed on the browser frame when the
-// user is in a multi-profile-enabled account. Stub for now. TODO(mirandac):
-// enable and fill in as part of multi-profile work.
+// Menu for the multi-profile button displayed on the browser frame.
 class ProfileMenuModel : public ui::SimpleMenuModel,
                          public ui::SimpleMenuModel::Delegate {
  public:
@@ -28,6 +24,19 @@ class ProfileMenuModel : public ui::SimpleMenuModel,
       int command_id, ui::Accelerator* accelerator) OVERRIDE;
   virtual void ExecuteCommand(int command_id) OVERRIDE;
 
+  enum {

[sky, 2011/07/08 21:01:44]

enums should be first in a section.

[sail, 2011/07/08 21:53:57]

Done.

+    COMMAND_PROFILE_NAME,
+    COMMAND_CHOOSE_AVATAR_ICON,
+    COMMAND_CUSTOMIZE_PROFILE,
+    COMMAND_DELETE_PROFILE,
+    COMMAND_CREATE_NEW_PROFILE,
+    COMMAND_SWITCH_PROFILE_MENU,
+    // The profiles submenu contains a menu item for each profile. For the i'th
+    // profile the command ID is COMMAND_SWITCH_TO_PROFILE + i. Since there can
+    // be any number of profiles this must be the last command id.
+    COMMAND_SWITCH_TO_PROFILE,
+  };
+
  private:
   Browser* browser_;
   scoped_ptr<ui::SimpleMenuModel> switch_profiles_sub_menu_model_;