Add the button style for combobox

ui/views/controls/combobox/combobox.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 "ui/views/controls/combobox/combobox.h"
 
 #include "base/logging.h"
 #include "base/strings/utf_string_conversions.h"
 #include "grit/ui_resources.h"
 #include "ui/base/accessibility/accessible_view_state.h"
 #include "ui/base/models/combobox_model.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/events/event.h"
 #include "ui/events/keycodes/keyboard_codes.h"
+#include "ui/gfx/animation/throb_animation.h"
 #include "ui/gfx/canvas.h"
+#include "ui/gfx/image/image.h"
 #include "ui/native_theme/native_theme.h"
 #include "ui/views/color_constants.h"
+#include "ui/views/controls/button/custom_button.h"
+#include "ui/views/controls/button/label_button.h"
 #include "ui/views/controls/combobox/combobox_listener.h"
 #include "ui/views/controls/focusable_border.h"
 #include "ui/views/controls/menu/menu_runner.h"
 #include "ui/views/controls/menu/submenu_view.h"
 #include "ui/views/controls/prefix_selector.h"
 #include "ui/views/ime/input_method.h"
 #include "ui/views/mouse_constants.h"
+#include "ui/views/painter.h"
 #include "ui/views/widget/widget.h"
 
 namespace views {
 
 namespace {
 
 // Menu border widths
 const int kMenuBorderWidthLeft = 1;
 const int kMenuBorderWidthTop = 1;
 const int kMenuBorderWidthRight = 1;
 
 // Limit how small a combobox can be.
 const int kMinComboboxWidth = 25;
 
 // Size of the combobox arrow margins
 const int kDisclosureArrowLeftPadding = 7;
 const int kDisclosureArrowRightPadding = 7;
+const int kDisclosureArrowButtonLeftPadding = 11;
+const int kDisclosureArrowButtonRightPadding = 12;
 
 // Define the id of the first item in the menu (since it needs to be > 0)
 const int kFirstMenuItemId = 1000;
 
 const SkColor kInvalidTextColor = SK_ColorWHITE;
 
 // Used to indicate that no item is currently selected by the user.
 const int kNoSelection = -1;
 
+const int kBodyButtonImages[] = IMAGE_GRID(IDR_COMBOBOX_BUTTON);
+const int kHoveredBodyButtonImages[] = IMAGE_GRID(IDR_COMBOBOX_BUTTON_H);
+const int kPressedBodyButtonImages[] = IMAGE_GRID(IDR_COMBOBOX_BUTTON_P);
+
+#define MENU_IMAGE_GRID(x) { \
+    x ## _MENU_TOP, x ## _MENU_CENTER, x ## _MENU_BOTTOM, }
+
+const int kMenuButtonImages[] = MENU_IMAGE_GRID(IDR_COMBOBOX_BUTTON);
+const int kHoveredMenuButtonImages[] = MENU_IMAGE_GRID(IDR_COMBOBOX_BUTTON_H);
+const int kPressedMenuButtonImages[] = MENU_IMAGE_GRID(IDR_COMBOBOX_BUTTON_P);
+
+#undef MENU_IMAGE_GRID
+
 // The background to use for invalid comboboxes.
 class InvalidBackground : public Background {
  public:
   InvalidBackground() {}
   virtual ~InvalidBackground() {}
 
   // Overridden from Background:
   virtual void Paint(gfx::Canvas* canvas, View* view) const OVERRIDE {
     gfx::Rect bounds(view->GetLocalBounds());
     // Inset by 2 to leave 1 empty pixel between background and border.
     bounds.Inset(2, 2, 2, 2);
     canvas->FillRect(bounds, kWarningColor);
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(InvalidBackground);
 };
 
+// The transparent button which holds a button state but is not rendered.
+class TransparentButton : public CustomButton {
+ public:
+  TransparentButton(ButtonListener* listener)
+      : CustomButton(listener) {
+    SetAnimationDuration(LabelButton::kHoverAnimationDurationMs);
+  }
+  virtual ~TransparentButton() {}
+
+  // CustomButton:
+  virtual void StateChanged() OVERRIDE {
+    parent()->SchedulePaintInRect(bounds());
+  }
+
+  // gfx::AnimationDelegate:
+  virtual void AnimationProgressed(const gfx::Animation* animation) OVERRIDE {
+    parent()->SchedulePaintInRect(bounds());
+  }
+
+  double GetAnimationValue() const {
+    return hover_animation_->GetCurrentValue();
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(TransparentButton);
+};
+
 // Returns the next or previous valid index (depending on |increment|'s value).
 // Skips separator or disabled indices. Returns -1 if there is no valid adjacent
 // index.
 int GetAdjacentIndex(ui::ComboboxModel* model, int increment, int index) {
   DCHECK(increment == -1 || increment == 1);
 
   index += increment;
   while (index >= 0 && index < model->GetItemCount()) {
     if (!model->IsItemSeparatorAt(index) || !model->IsItemEnabledAt(index))
       return index;
     index += increment;
   }
   return kNoSelection;
 }
 
+// Returns the image resource ids of an array for the body button.
+//
+// TODO(hajimehoshi): This function should return the images for the 'disabled'
+// status. (crbug/270052)
+//
+// TODO(hajimehoshi): Currently, |focused| is ignored. This should return the
+// images for the 'focused' status. (crbug/270052)
+const int* GetBodyButtonImageIds(bool focused, Button::ButtonState state,

[sky, 2013/12/04 16:34:05]

nit: wrap state to its own line.

[hajimehoshi, 2013/12/05 08:05:57]

Done.

+                                 size_t* num) {
+  DCHECK(num);
+  *num = 9;
+  switch (state) {
+    case Button::STATE_DISABLED:
+      return kBodyButtonImages;
+    case Button::STATE_NORMAL:
+      return kBodyButtonImages;
+    case Button::STATE_HOVERED:
+      return kHoveredBodyButtonImages;
+    case Button::STATE_PRESSED:
+      return kPressedBodyButtonImages;
+    default:
+      NOTREACHED();
+  }
+  return NULL;
+}
+
+// Returns the image resource ids of an array for the menu button.
+const int* GetMenuButtonImageIds(bool focused, Button::ButtonState state,

[sky, 2013/12/04 16:34:05]

nit: wrap state to its own line.

[hajimehoshi, 2013/12/05 08:05:57]

Done.

+                                 size_t* num) {
+  DCHECK(num);
+  *num = 3;
+  switch (state) {
+    case Button::STATE_DISABLED:
+      return kMenuButtonImages;
+    case Button::STATE_NORMAL:
+      return kMenuButtonImages;
+    case Button::STATE_HOVERED:
+      return kHoveredMenuButtonImages;
+    case Button::STATE_PRESSED:
+      return kPressedMenuButtonImages;
+    default:
+      NOTREACHED();
+  }
+  return NULL;
+}
+
+// Returns the images for the menu buttons.
+std::vector<const gfx::ImageSkia*> GetMenuButtonImages(
+    bool focused, Button::ButtonState state) {

[sky, 2013/12/04 16:34:05]

nit: wrap state to its own line.

[hajimehoshi, 2013/12/05 08:05:57]

Done.

+  const int* ids;
+  size_t num_ids;
+  ids = GetMenuButtonImageIds(focused, state, &num_ids);
+  std::vector<const gfx::ImageSkia*> images;
+  images.reserve(num_ids);
+  ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
+  for (size_t i = 0; i < num_ids; i++)
+    images.push_back(rb.GetImageSkiaNamed(ids[i]));
+  return images;
+}
+
+// Paints three images in a row at the given location.

[sky, 2013/12/04 16:34:05]

This isn't a row, but a column. Name this PaintImagesVertically and document the
center is stretched.

[hajimehoshi, 2013/12/05 08:05:57]

Done.

+void PaintThreeImages(gfx::Canvas* canvas,
+                      const gfx::ImageSkia& top_image,
+                      const gfx::ImageSkia& center_image,
+                      const gfx::ImageSkia& bottom_image,
+                      int x, int y, int width, int height) {
+  canvas->DrawImageInt(top_image,
+                       0, 0, top_image.width(), top_image.height(),
+                       x, y, width, top_image.height(), false);
+  y += top_image.height();
+  int center_height = height - top_image.height() - bottom_image.height();
+  canvas->DrawImageInt(center_image,
+                       0, 0, center_image.width(), center_image.height(),
+                       x, y, width, center_height, false);
+  y += center_height;
+  canvas->DrawImageInt(bottom_image,
+                       0, 0, bottom_image.width(), bottom_image.height(),
+                       x, y, width, bottom_image.height(), false);
+}
+
+// Paints the arrow button.
+void PaintArrowButton(
+    gfx::Canvas* canvas,
+    const std::vector<const gfx::ImageSkia*>& arrow_button_images,
+    int x, int height) {
+  PaintThreeImages(canvas,
+                   *arrow_button_images[0],
+                   *arrow_button_images[1],
+                   *arrow_button_images[2],
+                   x, 0, arrow_button_images[0]->width(), height);
+}
+
 }  // namespace
 
 // static
 const char Combobox::kViewClassName[] = "views/Combobox";
 
 ////////////////////////////////////////////////////////////////////////////////
 // Combobox, public:
 
 Combobox::Combobox(ui::ComboboxModel* model)
     : model_(model),
+      style_(STYLE_SHOW_DROP_DOWN_ON_CLICK),
       listener_(NULL),
       selected_index_(model_->GetDefaultIndex()),
       invalid_(false),
       text_border_(new FocusableBorder()),
       disclosure_arrow_(ui::ResourceBundle::GetSharedInstance().GetImageNamed(
           IDR_MENU_DROPARROW).ToImageSkia()),
-      dropdown_open_(false) {
+      dropdown_open_(false),
+      text_button_(NULL),

[sky, 2013/12/04 16:34:05]

Sorry for not noticing last time around. Why can't we always create these and
always use them? The only difference between the two modes is slight rendering
differences and keyboard handling, right?

[hajimehoshi, 2013/12/05 08:05:57]

Right.  Done.

I guess OnMousePressed, OnMouseDragged and OnGestureEvent are no longer needed
so I'll remove them.

+      arrow_button_(NULL) {
   model_->AddObserver(this);
   UpdateFromModel();
   set_focusable(true);
   set_border(text_border_);
+
+  // Initialize the button images.
+  Button::ButtonState button_states[] = {
+    Button::STATE_DISABLED,
+    Button::STATE_NORMAL,
+    Button::STATE_HOVERED,
+    Button::STATE_PRESSED,
+  };
+  for (int focused = 0; focused < 2; focused++) {
+    for (size_t state_index = 0; state_index < arraysize(button_states);
+         state_index++) {
+      Button::ButtonState state = button_states[state_index];
+      size_t num;
+      const int* ids = GetBodyButtonImageIds(focused, state, &num);
+      body_button_painters_[focused][state].reset(
+          Painter::CreateImageGridPainter(ids));
+      menu_button_images_[focused][state] = GetMenuButtonImages(focused, state);
+    }
+  }
 }
 
 Combobox::~Combobox() {
   model_->RemoveObserver(this);
 }
 
 // static
 const gfx::Font& Combobox::GetFont() {
   ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
   return rb.GetFont(ui::ResourceBundle::BaseFont);
 }
 
+void Combobox::SetStyle(Style style) {
+  if (style_ == style)
+    return;
+
+  style_ = style;
+  switch (style) {
+    case STYLE_SHOW_DROP_DOWN_ON_CLICK: {
+      // Recreate a FocusableBorder to reset the insets of the border.
+      // set_border with a new border will delete the existing border.
+      text_border_ = new FocusableBorder();
+      set_border(text_border_);
+
+      DCHECK(text_button_);
+      DCHECK(arrow_button_);
+      RemoveChildView(text_button_);

[sky, 2013/12/04 16:34:05]

delete text_button_; delete arrow_button_; does the right thing.

You also need to set text_button/arrow_button_ to NULL.

[hajimehoshi, 2013/12/05 08:05:57]

Done.

+      RemoveChildView(arrow_button_);
+      // Since RemoveChildView doesn't delete the given views, they should be
+      // deleted here.
+      scoped_ptr<View> view_to_be_deleted;
+      view_to_be_deleted.reset(text_button_);
+      view_to_be_deleted.reset(arrow_button_);
+      break;
+    }
+    case STYLE_NOTIFY_ON_CLICK: {
+      text_border_->SetInsets(8, 13, 8, 13);
+
+      DCHECK(!text_button_);
+      DCHECK(!arrow_button_);
+      text_button_ = new TransparentButton(this);
+      arrow_button_ = new TransparentButton(this);
+      text_button_->SetVisible(true);
+      arrow_button_->SetVisible(true);
+      text_button_->set_focusable(false);
+      arrow_button_->set_focusable(false);
+      AddChildView(text_button_);
+      AddChildView(arrow_button_);
+      break;
+    }
+  }
+
+  PreferredSizeChanged();
+}
+
 void Combobox::ModelChanged() {
   selected_index_ = std::min(0, model_->GetItemCount());
   UpdateFromModel();
   PreferredSizeChanged();
 }
 
 void Combobox::SetSelectedIndex(int index) {
   selected_index_ = index;
   SchedulePaint();
 }
@@ skipping 22 matching lines @@
     set_background(NULL);
   }
 }
 
 ui::TextInputClient* Combobox::GetTextInputClient() {
   if (!selector_)
     selector_.reset(new PrefixSelector(this));
   return selector_.get();
 }
 
+void Combobox::Layout() {
+  PrefixDelegate::Layout();
+  if (style_ != STYLE_NOTIFY_ON_CLICK)
+    return;
+
+  DCHECK(text_button_);
+  DCHECK(arrow_button_);
+  gfx::Insets insets = GetInsets();
+  int height = content_size_.height() + insets.height();
+  int text_button_width =
+      std::max(kMinComboboxWidth, content_size_.width()) + insets.width();

[sky, 2013/12/04 16:34:05]

I don't think you should use content_size_ here. It does not necessarily match
the actual bounds. Instead layout the arrow button aligned to the right side,
then give the textbutton the rest of the space. That way things work all the
time, even if less/more space is given.

[hajimehoshi, 2013/12/05 08:05:57]

Done.

+  text_button_->SetBounds(0, 0, text_button_width, height);
+  int arrow_button_x = text_button_width;
+  int arrow_button_width = GetDisclosureArrowLeftPadding() +
+      disclosure_arrow_->width() + GetDisclosureArrowRightPadding();
+  arrow_button_->SetBounds(arrow_button_x, 0, arrow_button_width, height);
+}
 
 bool Combobox::IsItemChecked(int id) const {
   return false;
 }
 
 bool Combobox::IsCommandEnabled(int id) const {
   return model()->IsItemEnabledAt(MenuCommandToIndex(id));
 }
 
 void Combobox::ExecuteCommand(int id) {
@@ skipping 25 matching lines @@
 // Combobox, View overrides:
 
 gfx::Size Combobox::GetPreferredSize() {
   if (content_size_.IsEmpty())
     UpdateFromModel();
 
   // The preferred size will drive the local bounds which in turn is used to set
   // the minimum width for the dropdown list.
   gfx::Insets insets = GetInsets();
   int total_width = std::max(kMinComboboxWidth, content_size_.width()) +
-      insets.width() + kDisclosureArrowLeftPadding +
-      disclosure_arrow_->width() + kDisclosureArrowRightPadding;
-
+      insets.width() + GetDisclosureArrowLeftPadding() +
+      disclosure_arrow_->width() + GetDisclosureArrowRightPadding();
   return gfx::Size(total_width, content_size_.height() + insets.height());
 }
 
 const char* Combobox::GetClassName() const {
   return kViewClassName;
 }
 
 bool Combobox::SkipDefaultKeyEventProcessing(const ui::KeyEvent& e) {
   // Escape should close the drop down list when it is active, not host UI.
   if (e.key_code() != ui::VKEY_ESCAPE ||
       e.IsShiftDown() || e.IsControlDown() || e.IsAltDown()) {
     return false;
   }
   return dropdown_open_;
 }
 
 bool Combobox::OnMousePressed(const ui::MouseEvent& mouse_event) {
   RequestFocus();
   const base::TimeDelta delta = base::Time::Now() - closed_time_;
   if (mouse_event.IsLeftMouseButton() &&
       (delta.InMilliseconds() > kMinimumMsBetweenButtonClicks)) {
     UpdateFromModel();
     ShowDropDownMenu(ui::MENU_SOURCE_MOUSE);
   }
-
   return true;
 }
 
 bool Combobox::OnMouseDragged(const ui::MouseEvent& mouse_event) {
   return true;
 }
 
 bool Combobox::OnKeyPressed(const ui::KeyEvent& e) {
   // TODO(oshima): handle IME.
   DCHECK_EQ(e.type(), ui::ET_KEY_PRESSED);
@@ skipping 31 matching lines @@
     case ui::VKEY_HOME:
     case ui::VKEY_PRIOR:  // Page up.
       new_index = GetAdjacentIndex(model(), 1, -1);
       break;
 
     // Move to the previous item if any.
     case ui::VKEY_UP:
       new_index = GetAdjacentIndex(model(), -1, selected_index_);
       break;
 
+    // Click the button only when the button style mode.
+    case ui::VKEY_SPACE:
+      if (style_ == STYLE_NOTIFY_ON_CLICK) {
+        // When pressing space, the click event will be raised after the key is
+        // released.
+        text_button_->SetState(Button::STATE_PRESSED);
+      }
+      break;
+
+    // Click the button only when the button style mode.
+    case ui::VKEY_RETURN:
+      HandleClickEvent();
+      break;
+
     default:
       return false;
   }
 
   if (show_menu) {
     UpdateFromModel();
     ShowDropDownMenu(ui::MENU_SOURCE_KEYBOARD);
   } else if (new_index != selected_index_ && new_index != kNoSelection) {
     DCHECK(!model()->IsItemSeparatorAt(new_index));
     selected_index_ = new_index;
     OnSelectionChanged();
   }
 
   return true;
 }
 
 bool Combobox::OnKeyReleased(const ui::KeyEvent& e) {
-  return false;  // crbug.com/127520
+  if (style_ != STYLE_NOTIFY_ON_CLICK)
+    return false;  // crbug.com/127520
+
+  if (e.key_code() == ui::VKEY_SPACE)
+    HandleClickEvent();
+
+  return false;
 }
 
 void Combobox::OnGestureEvent(ui::GestureEvent* gesture) {
   if (gesture->type() == ui::ET_GESTURE_TAP) {
     UpdateFromModel();
     ShowDropDownMenu(ui::MENU_SOURCE_TOUCH);
     gesture->StopPropagation();
     return;
   }
   View::OnGestureEvent(gesture);
 }
 
 void Combobox::OnPaint(gfx::Canvas* canvas) {
-  OnPaintBackground(canvas);
-  PaintText(canvas);
-  OnPaintBorder(canvas);
+  switch (style_) {
+    case STYLE_SHOW_DROP_DOWN_ON_CLICK: {
+      OnPaintBackground(canvas);
+      PaintText(canvas);
+      OnPaintBorder(canvas);
+      break;
+    }
+    case STYLE_NOTIFY_ON_CLICK: {
+      PaintButtons(canvas);
+      PaintText(canvas);
+      break;

[sky, 2013/12/04 16:34:05]

I believe you need to deal with focus here.

[hajimehoshi, 2013/12/05 08:05:57]

I intend to create another CL to add images to render the focused buttons. We
think that focus border should be put on the whole of the combobox, not on each
button, so we prepared the other images which are not like buttons. The reason
why I didn't include them is that including them make this CL too big.

+    }
+  }
 }
 
 void Combobox::OnFocus() {
   GetInputMethod()->OnFocus();
   text_border_->set_has_focus(true);
   View::OnFocus();
 }
 
 void Combobox::OnBlur() {
   GetInputMethod()->OnBlur();
   if (selector_)
     selector_->OnViewBlur();
   text_border_->set_has_focus(false);
 }
 
 void Combobox::GetAccessibleState(ui::AccessibleViewState* state) {
   state->role = ui::AccessibilityTypes::ROLE_COMBOBOX;
   state->name = accessible_name_;
   state->value = model_->GetItemAt(selected_index_);
   state->index = selected_index_;
   state->count = model_->GetItemCount();
 }
 
 void Combobox::OnModelChanged() {
   ModelChanged();
 }
 
+void Combobox::ButtonPressed(Button* sender, const ui::Event& event) {
+  DCHECK(style_ == STYLE_NOTIFY_ON_CLICK);
+  DCHECK(text_button_);
+  DCHECK(arrow_button_);
+  if (sender == text_button_) {
+    HandleClickEvent();
+  } else {
+    DCHECK_EQ(arrow_button_, sender);
+    // TODO(hajimehoshi): Fix the problem that the arrow button blinks when
+    // cliking this while the dropdown menu is opened.
+    const base::TimeDelta delta = base::Time::Now() - closed_time_;
+    if (delta.InMilliseconds() <= kMinimumMsBetweenButtonClicks)
+      return;
+
+    UpdateFromModel();

[sky, 2013/12/04 16:34:05]

Why are you invoking UpdateFromModel here?

[hajimehoshi, 2013/12/05 08:05:57]

Sorry but this wasn't needed. Done.

+    ui::MenuSourceType source_type = ui::MENU_SOURCE_MOUSE;
+    if (event.IsKeyEvent())
+      source_type = ui::MENU_SOURCE_KEYBOARD;
+    else if (event.IsGestureEvent() || event.IsTouchEvent())
+      source_type = ui::MENU_SOURCE_TOUCH;
+    ShowDropDownMenu(source_type);
+  }
+}
+
 void Combobox::UpdateFromModel() {
   int max_width = 0;
   const gfx::Font& font = Combobox::GetFont();
 
   MenuItemView* menu = new MenuItemView(this);
   // MenuRunner owns |menu|.
   dropdown_list_menu_runner_.reset(new MenuRunner(menu));
 
   int num_items = model()->GetItemCount();
   for (int i = 0; i < num_items; ++i) {
@@ skipping 31 matching lines @@
   SkColor text_color = invalid() ? kInvalidTextColor :
       GetNativeTheme()->GetSystemColor(
           ui::NativeTheme::kColorId_LabelEnabledColor);
 
   DCHECK_GE(selected_index_, 0);
   DCHECK_LT(selected_index_, model()->GetItemCount());
   if (selected_index_ < 0 || selected_index_ > model()->GetItemCount())
     selected_index_ = 0;
   string16 text = model()->GetItemAt(selected_index_);
 
-  int disclosure_arrow_offset = width() - disclosure_arrow_->width()
-      - kDisclosureArrowLeftPadding - kDisclosureArrowRightPadding;
+  int disclosure_arrow_offset = width() - disclosure_arrow_->width() -
+      GetDisclosureArrowLeftPadding() - GetDisclosureArrowRightPadding();
 
   const gfx::Font& font = Combobox::GetFont();
   int text_width = font.GetStringWidth(text);
   if ((text_width + insets.width()) > disclosure_arrow_offset)
     text_width = disclosure_arrow_offset - insets.width();
 
   gfx::Rect text_bounds(x, y, text_width, text_height);
   AdjustBoundsForRTLUI(&text_bounds);
   canvas->DrawStringInt(text, font, text_color, text_bounds);
 
-  gfx::Rect arrow_bounds(disclosure_arrow_offset + kDisclosureArrowLeftPadding,
+  int arrow_x = disclosure_arrow_offset + GetDisclosureArrowLeftPadding();
+  gfx::Rect arrow_bounds(arrow_x,
                          height() / 2 - disclosure_arrow_->height() / 2,
                          disclosure_arrow_->width(),
                          disclosure_arrow_->height());
   AdjustBoundsForRTLUI(&arrow_bounds);
 
   SkPaint paint;
   // This makes the arrow subtractive.
   if (invalid())
     paint.setXfermodeMode(SkXfermode::kDstOut_Mode);
   canvas->DrawImageInt(*disclosure_arrow_, arrow_bounds.x(), arrow_bounds.y(),
                        paint);
 
   canvas->Restore();
 }
 
+void Combobox::PaintButtons(gfx::Canvas* canvas) {
+  DCHECK(style_ == STYLE_NOTIFY_ON_CLICK);
+  DCHECK(text_button_);
+  DCHECK(arrow_button_);
+
+  canvas->Save();

[sky, 2013/12/04 16:34:05]

Use ScopedCanvas as necessary in here.

[hajimehoshi, 2013/12/05 08:05:57]

Done.

+  if (base::i18n::IsRTL()) {
+    canvas->Translate(gfx::Vector2d(width(), 0));
+    canvas->Scale(-1, 1);
+  }
+
+  bool focused = HasFocus();
+  const std::vector<const gfx::ImageSkia*>& arrow_button_images =
+      menu_button_images_[focused][
+          arrow_button_->state() == Button::STATE_HOVERED ?
+          Button::STATE_NORMAL : arrow_button_->state()];
+
+  int text_button_width = width() - arrow_button_images[0]->width();
+  int text_button_hover_alpha =
+      text_button_->state() == Button::STATE_PRESSED ? 0 :
+      static_cast<int>(static_cast<TransparentButton*>(text_button_)->
+                       GetAnimationValue() * 255);
+  if (text_button_hover_alpha < 255) {
+    canvas->SaveLayerAlpha(255 - text_button_hover_alpha);
+    Painter* text_button_painter =
+        body_button_painters_[focused][
+            text_button_->state() == Button::STATE_HOVERED ?
+            Button::STATE_NORMAL : text_button_->state()].get();
+    Painter::PaintPainterAt(canvas, text_button_painter,
+                            gfx::Rect(0, 0, text_button_width, height()));
+    canvas->Restore();
+  }
+  if (0 < text_button_hover_alpha) {
+    canvas->SaveLayerAlpha(text_button_hover_alpha);
+    Painter* text_button_hovered_painter =
+        body_button_painters_[focused][Button::STATE_HOVERED].get();
+    Painter::PaintPainterAt(canvas, text_button_hovered_painter,
+                            gfx::Rect(0, 0, text_button_width, height()));
+    canvas->Restore();
+  }
+
+  int arrow_button_x = text_button_width;
+  int arrow_button_hover_alpha =
+      arrow_button_->state() == Button::STATE_PRESSED ? 0 :
+      static_cast<int>(static_cast<TransparentButton*>(arrow_button_)->
+                       GetAnimationValue() * 255);
+  if (arrow_button_hover_alpha < 255) {
+    canvas->SaveLayerAlpha(255 - arrow_button_hover_alpha);
+    PaintArrowButton(canvas, arrow_button_images, arrow_button_x, height());
+    canvas->Restore();
+  }
+  if (0 < arrow_button_hover_alpha) {
+    canvas->SaveLayerAlpha(arrow_button_hover_alpha);
+    const std::vector<const gfx::ImageSkia*>& arrow_button_hovered_images =
+        menu_button_images_[focused][Button::STATE_HOVERED];
+    PaintArrowButton(canvas, arrow_button_hovered_images,
+                     arrow_button_x, height());
+    canvas->Restore();
+  }
+
+  canvas->Restore();
+}
+
 void Combobox::ShowDropDownMenu(ui::MenuSourceType source_type) {
   if (!dropdown_list_menu_runner_.get())
     UpdateFromModel();
 
   // Extend the menu to the width of the combobox.
   MenuItemView* menu = dropdown_list_menu_runner_->GetMenu();
   SubmenuView* submenu = menu->CreateSubmenu();
-  submenu->set_minimum_preferred_width(size().width() -
-                                (kMenuBorderWidthLeft + kMenuBorderWidthRight));
+  submenu->set_minimum_preferred_width(
+      size().width() - (kMenuBorderWidthLeft + kMenuBorderWidthRight));
 
   gfx::Rect lb = GetLocalBounds();
   gfx::Point menu_position(lb.origin());
 
   // Inset the menu's requested position so the border of the menu lines up
   // with the border of the combobox.
   menu_position.set_x(menu_position.x() + kMenuBorderWidthLeft);
   menu_position.set_y(menu_position.y() + kMenuBorderWidthTop);
   lb.set_width(lb.width() - (kMenuBorderWidthLeft + kMenuBorderWidthRight));
 
   View::ConvertPointToScreen(this, &menu_position);
   if (menu_position.x() < 0)
-      menu_position.set_x(0);
+    menu_position.set_x(0);
 
   gfx::Rect bounds(menu_position, lb.size());
 
+  Button::ButtonState original_state;
+  if (arrow_button_) {
+    original_state = arrow_button_->state();
+    arrow_button_->SetState(Button::STATE_PRESSED);
+  }
   dropdown_open_ = true;
   if (dropdown_list_menu_runner_->RunMenuAt(GetWidget(), NULL, bounds,
           MenuItemView::TOPLEFT, source_type, MenuRunner::COMBOBOX) ==
-      MenuRunner::MENU_DELETED)
+      MenuRunner::MENU_DELETED) {
     return;
+  }
   dropdown_open_ = false;
+  if (arrow_button_) {
+    arrow_button_->SetState(original_state);

[sky, 2013/12/04 16:34:05]

nit: no {}

[hajimehoshi, 2013/12/05 08:05:57]

Done.

+  }
   closed_time_ = base::Time::Now();
 
   // Need to explicitly clear mouse handler so that events get sent
   // properly after the menu finishes running. If we don't do this, then
   // the first click to other parts of the UI is eaten.
   SetMouseHandler(NULL);
 }
 
 void Combobox::OnSelectionChanged() {
   if (listener_)
     listener_->OnSelectedIndexChanged(this);
   NotifyAccessibilityEvent(ui::AccessibilityTypes::EVENT_VALUE_CHANGED, false);
   SchedulePaint();
 }
 
 int Combobox::MenuCommandToIndex(int menu_command_id) const {
   // (note that the id received is offset by kFirstMenuItemId)
   // Revert menu ID offset to map back to combobox model.
   int index = menu_command_id - kFirstMenuItemId;
   DCHECK_LT(index, model()->GetItemCount());
   return index;
 }
 
+int Combobox::GetDisclosureArrowLeftPadding() const {
+  switch (style_) {
+    case STYLE_SHOW_DROP_DOWN_ON_CLICK:
+      return kDisclosureArrowLeftPadding;
+    case STYLE_NOTIFY_ON_CLICK:
+      return kDisclosureArrowButtonLeftPadding;
+  }
+  NOTREACHED();
+  return 0;
+}
+
+int Combobox::GetDisclosureArrowRightPadding() const {
+  switch (style_) {
+    case STYLE_SHOW_DROP_DOWN_ON_CLICK:
+      return kDisclosureArrowRightPadding;
+    case STYLE_NOTIFY_ON_CLICK:
+      return kDisclosureArrowButtonRightPadding;
+  }
+  NOTREACHED();
+  return 0;
+}
+
+void Combobox::HandleClickEvent() {
+  if (style_ != STYLE_NOTIFY_ON_CLICK)
+    return;
+
+  if (listener_)
+    listener_->OnComboboxTextButtonClicked(this);
+}
+
 }  // namespace views