Action box menu

chrome/browser/ui/views/action_box_menu_item_view.cc

+// Copyright (c) 2012 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/ui/views/action_box_menu_item_view.h"
+
+// ActionBoxMenuItemView -------------------------------------------------------
+
+ActionBoxMenuItemView::ActionBoxMenuItemView(views::MenuDelegate* delegate)
+    : views::MenuItemView(delegate) {
+}
+
+void ActionBoxMenuItemView::Layout() {
+  MenuItemView::Layout();

[Aaron Boodman, 2012/06/12 05:53:44]

I think the child layout is unnecessary since we're doing our own custom layout
anyway.

[yefimt, 2012/06/13 01:24:21]

Not sure it is true, we layout only icon view, and let the base class to do the
rest

On 2012/06/12 05:53:44, Aaron Boodman wrote:

+  if (!has_children())
+    return;
+
+  View* child = child_at(0);
+  gfx::Size size = child->GetPreferredSize();

[Aaron Boodman, 2012/06/12 05:53:44]

child->SetPosition(Position(0, 0));
child->SizeToPreferredSize();

[yefimt, 2012/06/13 01:24:21]

Done.

+  child->SetBounds(0, 0, size.width(), size.height());
+}
+
+ActionBoxMenuItemView::ActionBoxMenuItemView(MenuItemView* parent,
+                                             int command,
+                                             MenuItemView::Type type)
+    : MenuItemView(parent, command, type) {
+}
+
+ActionBoxMenuItemView::~ActionBoxMenuItemView() {
+}
+
+void ActionBoxMenuItemView::OnPaint(gfx::Canvas* canvas) {
+  MenuItemView::OnPaint(canvas);

[Aaron Boodman, 2012/06/12 05:53:44]

No need to override?

[yefimt, 2012/06/13 01:24:21]

Yes, removed it after it was uploaded
On 2012/06/12 05:53:44, Aaron Boodman wrote:

+}
+
+views::MenuItemView* ActionBoxMenuItemView::AllocateMenuItemView(
+    views::MenuItemView* parent, int item_id, Type type) {
+  return new ActionBoxMenuItemView(parent, item_id, type);
+}