Continue work on requestAutocomplete dialog

chrome/browser/ui/views/autofill/autofill_dialog_views.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/autofill/autofill_dialog_views.h"
 
 #include <utility>
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/ui/autofill/autofill_dialog_controller.h"
 #include "chrome/browser/ui/views/constrained_window_views.h"
+#include "grit/theme_resources.h"
 #include "third_party/skia/include/core/SkColor.h"
+#include "ui/base/resource/resource_bundle.h"
 #include "ui/views/border.h"
 #include "ui/views/controls/button/checkbox.h"
-#include "ui/views/controls/button/menu_button.h"
+#include "ui/views/controls/button/image_button.h"
 #include "ui/views/controls/combobox/combobox.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/controls/menu/menu_model_adapter.h"
 #include "ui/views/controls/menu/menu_runner.h"
 #include "ui/views/controls/separator.h"
 #include "ui/views/controls/textfield/textfield.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/grid_layout.h"
 #include "ui/views/layout/layout_constants.h"
 
 namespace autofill {
 
 namespace {
 
 // Returns a label that describes a details section.
 views::Label* CreateDetailsSectionLabel(const string16& text) {
   views::Label* label = new views::Label(text);
   label->SetHorizontalAlignment(gfx::ALIGN_RIGHT);
   label->SetFont(label->font().DeriveFont(0, gfx::Font::BOLD));
   // TODO(estade): this should be made to match the native textfield top
   // inset. It's hard to get at, so for now it's hard-coded.
   label->set_border(views::Border::CreateEmptyBorder(4, 0, 0, 0));
   return label;
 }
 
-// Creates a view that packs a label on the left and some related controls
-// on the right.
-views::View* CreateSectionContainer(const string16& label,
-                                    views::View* controls) {
-  views::View* container = new views::View();
-  views::GridLayout* layout = new views::GridLayout(container);
-  container->SetLayoutManager(layout);
+}  // namespace
+
+// AutofillDialogViews::SectionContainer ---------------------------------------
+
+AutofillDialogViews::SectionContainer::SectionContainer(const string16& label,
+                                   views::View* controls,

[sky, 2013/01/08 05:05:12]

nit: fix indentation.

[Evan Stade, 2013/01/08 21:41:14]

Done.

+                                   views::Button* proxy_button)
+    : proxy_button_(proxy_button),
+      forward_mouse_events_(false) {
+  set_notify_enter_exit_on_child(true);
+
+  views::GridLayout* layout = new views::GridLayout(this);
+  SetLayoutManager(layout);
 
   const int kColumnSetId = 0;
   views::ColumnSet* column_set = layout->AddColumnSet(kColumnSetId);
   // TODO(estade): pull out these constants, and figure out better values
   // for them.
   column_set->AddColumn(views::GridLayout::FILL,
                         views::GridLayout::LEADING,
                         0,
                         views::GridLayout::FIXED,
                         180,
                         0);
   column_set->AddPaddingColumn(0, 15);
   column_set->AddColumn(views::GridLayout::FILL,
                         views::GridLayout::LEADING,
                         0,
                         views::GridLayout::FIXED,
                         300,
                         0);
 
   layout->StartRow(0, kColumnSetId);
   layout->AddView(CreateDetailsSectionLabel(label));
   layout->AddView(controls);
-  return container;
 }
 
-}  // namespace
+AutofillDialogViews::SectionContainer::~SectionContainer() {}
+
+void AutofillDialogViews::SectionContainer::SetForwardMouseEvents(

[sky, 2013/01/08 05:05:12]

Do you really need this? Can forward_mouse_events_ always be true?

[Evan Stade, 2013/01/08 21:41:14]

it's false when in edit mode. In that case, you have to actually click on the
[v] button.

+    bool forward) {
+  forward_mouse_events_ = forward;
+  if (!forward)
+    set_background(NULL);
+}
+
+void AutofillDialogViews::SectionContainer::OnMouseEntered(
+    const ui::MouseEvent& event) {
+  if (!forward_mouse_events_)
+    return;
+
+  // TODO(estade): use the correct color.
+  set_background(views::Background::CreateSolidBackground(SK_ColorLTGRAY));
+  proxy_button_->OnMouseEntered(ProxyEvent(event));
+  SchedulePaint();
+}
+
+void AutofillDialogViews::SectionContainer::OnMouseExited(
+    const ui::MouseEvent& event) {
+  if (!forward_mouse_events_)
+    return;
+
+  set_background(NULL);
+  proxy_button_->OnMouseExited(ProxyEvent(event));

[sky, 2013/01/08 05:05:12]

Is proxy_button_ a descendant? Does SectionContainer contain any views that want
mouse events, like the radio box titled 'Use as shipping address' in the screen
shot?

[Evan Stade, 2013/01/08 21:41:14]

yes it is, although I don't think it has to be for this to work. The radio
button is a descendant of SectinContainer, and it still works.

+  SchedulePaint();
+}
+
+bool AutofillDialogViews::SectionContainer::OnMousePressed(
+    const ui::MouseEvent& event) {
+  if (!forward_mouse_events_)
+    return false;
+
+  return proxy_button_->OnMousePressed(ProxyEvent(event));
+}
+
+void AutofillDialogViews::SectionContainer::OnMouseReleased(
+    const ui::MouseEvent& event) {
+  if (!forward_mouse_events_)
+    return;
+
+  proxy_button_->OnMouseReleased(ProxyEvent(event));
+}
+
+// static
+ui::MouseEvent AutofillDialogViews::SectionContainer::ProxyEvent(
+    const ui::MouseEvent& event) {
+  ui::MouseEvent event_copy = event;
+  event_copy.set_location(gfx::Point());
+  return event_copy;
+}
+
+// AutofillDialogView ----------------------------------------------------------
 
 // static
 AutofillDialogView* AutofillDialogView::Create(
     AutofillDialogController* controller) {
   return new AutofillDialogViews(controller);
 }
 
+// AutofillDialogViews ---------------------------------------------------------
+
 AutofillDialogViews::AutofillDialogViews(AutofillDialogController* controller)
     : controller_(controller),
       did_submit_(false),
       window_(NULL),
       contents_(NULL),
       email_(SECTION_EMAIL),
       cc_(SECTION_CC),
       billing_(SECTION_BILLING),
       shipping_(SECTION_SHIPPING) {
   DCHECK(controller);
@@ skipping 84 matching lines @@
   return true;
 }
 
 bool AutofillDialogViews::Accept() {
   did_submit_ = true;
   return true;
 }
 
 void AutofillDialogViews::ButtonPressed(views::Button* sender,
                                         const ui::Event& event) {
-  DCHECK_EQ(sender, use_billing_for_shipping_);
-  shipping_.container->SetVisible(!use_billing_for_shipping_->checked());
-  GetWidget()->SetSize(GetWidget()->non_client_view()->GetPreferredSize());
-}
+  if (sender == use_billing_for_shipping_) {
+    shipping_.container->SetVisible(!use_billing_for_shipping_->checked());
+    GetWidget()->SetSize(GetWidget()->non_client_view()->GetPreferredSize());
+  } else {
+    // TODO(estade): Should the menu be shown on mouse down?
+    DetailsGroup* group =
+        sender == email_.suggested_button ? &email_ :
+        sender == cc_.suggested_button ? &cc_ :
+        sender == billing_.suggested_button ? &billing_ :
+        sender == shipping_.suggested_button ? &shipping_ : NULL;
+    DCHECK(group);
 
-void AutofillDialogViews::OnMenuButtonClicked(views::View* source,
-                                              const gfx::Point& point) {
-  DetailsGroup* group =
-      source == email_.suggested_button ? &email_ :
-      source == cc_.suggested_button ? &cc_ :
-      source == billing_.suggested_button ? &billing_ :
-      source == shipping_.suggested_button ? &shipping_ : NULL;
-  DCHECK(group);
-  views::MenuModelAdapter adapter(
-      controller_->MenuModelForSection(group->section));
-  menu_runner_.reset(new views::MenuRunner(adapter.CreateMenu()));
+    views::MenuModelAdapter adapter(
+        controller_->MenuModelForSection(group->section));
+    menu_runner_.reset(new views::MenuRunner(adapter.CreateMenu()));
 
-  // Ignore the result since we don't need to handle a deleted menu specially.
-  ignore_result(
-      menu_runner_->RunMenuAt(source->GetWidget(),
-                              group->suggested_button,
-                              gfx::Rect(point, gfx::Size()),
-                              views::MenuItemView::TOPRIGHT,
-                              0));
+    // Ignore the result since we don't need to handle a deleted menu specially.
+    ignore_result(
+        menu_runner_->RunMenuAt(sender->GetWidget(),
+                                NULL,
+                                group->suggested_button->GetBoundsInScreen(),
+                                views::MenuItemView::TOPRIGHT,
+                                0));
+  }
 }
 
 void AutofillDialogViews::ContentsChanged(views::Textfield* sender,
                                           const string16& new_contents) {
   // TODO(estade): work for not billing.
   for (TextfieldMap::iterator iter = billing_.textfields.begin();
        iter != billing_.textfields.end();
        ++iter) {
     if (iter->second == sender) {
       controller_->UserEditedInput(iter->first,
@@ skipping 51 matching lines @@
 
     layout->AddView(label);
   }
 
   layout->StartRow(0, single_column_set);
   layout->AddView(CreateIntroContainer());
 
   layout->StartRowWithPadding(0, single_column_set,
                               0, views::kUnrelatedControlVerticalSpacing);
   layout->AddView(CreateDetailsContainer());
-
-  // Separator.
-  layout->StartRowWithPadding(0, single_column_set,
-                              0, views::kUnrelatedControlVerticalSpacing);
-  layout->AddView(new views::Separator());
-
-  // Wallet checkbox.
-  layout->StartRowWithPadding(0, single_column_set,
-                              0, views::kRelatedControlVerticalSpacing);
-  layout->AddView(new views::Checkbox(controller_->WalletOptionText()));
 }
 
 views::View* AutofillDialogViews::CreateIntroContainer() {
   views::View* view = new views::View();
   view->SetLayoutManager(
       new views::BoxLayout(views::BoxLayout::kHorizontal, 0, 0, 0));
 
   std::pair<string16, string16> intro_text_parts =
       controller_->GetIntroTextParts();
 
@@ skipping 28 matching lines @@
 
   return view;
 }
 
 void AutofillDialogViews::CreateDetailsSection(DialogSection section) {
   DCHECK_NE(SECTION_CC, section);
   DCHECK_NE(SECTION_BILLING, section);
 
   // Inputs container (manual inputs + combobox).
   views::View* inputs_container = CreateInputsContainer(section);
-  // Container (holds label + inputs).
-  views::View* container = CreateSectionContainer(
-      controller_->LabelForSection(section), inputs_container);
 
   DetailsGroup* group = GroupForSection(section);
-  group->container = container;
+  // Container (holds label + inputs).
+  group->container = new SectionContainer(
+      controller_->LabelForSection(section),
+      inputs_container,
+      group->suggested_button);
+  UpdateDetailsGroupState(*group);
 }
 
 void AutofillDialogViews::CreateBillingSection() {
   views::View* billing = new views::View();
   billing->SetLayoutManager(new views::BoxLayout(
       views::BoxLayout::kVertical, 0, 0,
       views::kRelatedControlVerticalSpacing));
 
   static const DialogSection sections[] = { SECTION_CC, SECTION_BILLING };
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(sections); ++i) {
     // Inputs container (manual inputs + combobox).
     views::View* inputs_container = CreateInputsContainer(sections[i]);
     billing->AddChildView(inputs_container);
   }
 
   use_billing_for_shipping_ =
       new views::Checkbox(controller_->UseBillingForShippingText());
   use_billing_for_shipping_->SetChecked(true);
   use_billing_for_shipping_->set_listener(this);
   billing->AddChildView(use_billing_for_shipping_);
 
   // Container (holds label + inputs).
-  views::View* container = CreateSectionContainer(
-      controller_->LabelForSection(SECTION_BILLING), billing);
-  billing_.container = container;
+  billing_.container = new SectionContainer(
+      controller_->LabelForSection(SECTION_BILLING),
+      billing,
+      billing_.suggested_button);
+  UpdateDetailsGroupState(billing_);
+  UpdateDetailsGroupState(cc_);
 }
 
 views::View* AutofillDialogViews::CreateInputsContainer(DialogSection section) {
   views::View* inputs_container = new views::View();
   views::GridLayout* layout = new views::GridLayout(inputs_container);
   inputs_container->SetLayoutManager(layout);
 
   int kColumnSetId = 0;
   views::ColumnSet* column_set = layout->AddColumnSet(kColumnSetId);
   column_set->AddColumn(views::GridLayout::FILL,
@@ skipping 16 matching lines @@
   info_view->SetLayoutManager(
       new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0));
   views::View* manual_inputs = InitInputsView(section);
   info_view->AddChildView(manual_inputs);
   views::Label* suggested_info = new views::Label();
   suggested_info->SetHorizontalAlignment(gfx::ALIGN_LEFT);
   info_view->AddChildView(suggested_info);
   layout->AddView(info_view);
 
   // TODO(estade): Fix the appearance of this button.
-  views::MenuButton* menu_button =
-      new views::MenuButton(NULL, string16(), this, true);
+  views::ImageButton* menu_button = new views::ImageButton(this);
+  ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
+  menu_button->SetImage(views::CustomButton::STATE_NORMAL,
+      rb.GetImageSkiaNamed(IDR_AUTOFILL_DIALOG_MENU_BUTTON));
+  menu_button->SetImage(views::CustomButton::STATE_PRESSED,
+      rb.GetImageSkiaNamed(IDR_AUTOFILL_DIALOG_MENU_BUTTON_P));
   layout->AddView(menu_button);
 
   DetailsGroup* group = GroupForSection(section);
   group->suggested_button = menu_button;
   group->manual_input = manual_inputs;
   group->suggested_info = suggested_info;
-  UpdateDetailsGroupState(*group);
   return inputs_container;
 }
 
 // TODO(estade): we should be using Chrome-style constrained window padding
 // values.
 views::View* AutofillDialogViews::InitInputsView(DialogSection section) {
   const DetailInputs& inputs = controller_->RequestedFieldsForSection(section);
   TextfieldMap* textfields = &GroupForSection(section)->textfields;
   ComboboxMap* comboboxes = &GroupForSection(section)->comboboxes;
 
@@ skipping 51 matching lines @@
       layout->AddView(field);
     }
   }
 
   return view;
 }
 
 void AutofillDialogViews::UpdateDetailsGroupState(const DetailsGroup& group) {
   string16 suggestion_text =
       controller_->SuggestionTextForSection(group.section);
-  group.manual_input->SetVisible(suggestion_text.empty());
-  group.suggested_info->SetVisible(!suggestion_text.empty());
+  bool show_suggestions = !suggestion_text.empty();
+  group.manual_input->SetVisible(!show_suggestions);
+  group.suggested_info->SetVisible(show_suggestions);
   group.suggested_info->SetText(suggestion_text);
+
+  if (group.container)
+    group.container->SetForwardMouseEvents(show_suggestions);
+
   if (GetWidget())
     GetWidget()->SetSize(GetWidget()->non_client_view()->GetPreferredSize());
 }
 
 AutofillDialogViews::DetailsGroup* AutofillDialogViews::
     GroupForSection(DialogSection section) {
   switch (section) {
     case SECTION_EMAIL:
       return &email_;
     case SECTION_CC:
@@ skipping 11 matching lines @@
 AutofillDialogViews::DetailsGroup::DetailsGroup(DialogSection section)
     : section(section),
       container(NULL),
       manual_input(NULL),
       suggested_info(NULL),
       suggested_button(NULL) {}
 
 AutofillDialogViews::DetailsGroup::~DetailsGroup() {}
 
 }  // namespace autofill