Add profile selector menu to app list.

ui/base/models/avatar_menu_item_model.h

+// Copyright (c) 2012 The Chromium Authors. All rights reserved.

[tapted, 2013/07/29 01:59:10]

nit: Year can be 2013 (and no "(c)" )

[calamity, 2013/07/30 08:42:34]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef UI_BASE_AVATAR_MENU_ITEM_MODEL_H_
+#define UI_BASE_AVATAR_MENU_ITEM_MODEL_H_

[koz (OOO until 15th September), 2013/07/29 00:34:34]

should be UI_BASE_MODELS_AVATAR_MENU_ITEM_MODEL_H_

[calamity, 2013/07/30 08:42:34]

Done.

+
+#include "base/basictypes.h"
+#include "ui/gfx/image/image.h"
+
+// Represents an item in the AvatarMenuModel.
+struct AvatarMenuItemModel {

[tapted, 2013/07/29 01:59:10]

This guy should go in namespace ui - that seems to be the Way. Also you need to
declare it with UI_EXPORT, i.e.

class UI_EXPORT AvatarMenuItemModel { .. }

and #include "ui/base/ui_export.h"

[calamity, 2013/07/30 08:42:34]

Done.

+AvatarMenuItemModel(size_t model_index, const gfx::Image& icon)

[koz (OOO until 15th September), 2013/07/29 00:34:34]

This all needs to be indented two spaces.

[calamity, 2013/07/30 08:42:34]

Done.

+    : icon(icon),
+      active(false),
+      signed_in(false),
+      signin_required(false),
+      model_index(model_index) {}
+~AvatarMenuItemModel() {}

[tapted, 2013/07/29 01:59:10]

I think clang will warn at you for having a non-trivial destructor in a header
file... The constructor & destructor should be moved to a .cc 

[calamity, 2013/07/30 08:42:34]

Done.

+
+// The icon to be displayed next to the item.
+gfx::Image icon;
+
+// Whether or not the current browser is using this profile.
+bool active;
+
+// The name of this profile.
+string16 name;

[tapted, 2013/07/29 01:59:10]

nit: #include "base/strings/string16.h" for string16

[calamity, 2013/07/30 08:42:34]

Done.

+
+// A string representing the sync state of the profile.
+string16 sync_state;
+
+// Whether or not the current profile is signed in. If true, |sync_state| is
+// expected to be the email of the signed in user.
+bool signed_in;
+
+// Whether or not the current profile requires sign-in before use.
+bool signin_required;
+
+// The index in the |profile_cache| that this Item represents.
+size_t model_index;
+};
+
+#endif  // UI_BASE_AVATAR_MENU_ITEM_MODEL_H_