requestAutocomplete: move billing/shipping checkbox to shipping section

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"
@@ skipping 229 matching lines @@
 }
 
 bool AutofillDialogViews::Accept() {
   did_submit_ = true;
   return true;
 }
 
 void AutofillDialogViews::ButtonPressed(views::Button* sender,
                                         const ui::Event& event) {
   if (sender == use_billing_for_shipping_) {
-    shipping_.container->SetVisible(!use_billing_for_shipping_->checked());
-    GetWidget()->SetSize(GetWidget()->non_client_view()->GetPreferredSize());
+    UpdateDetailsGroupState(shipping_);
   } 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);
 
     views::MenuModelAdapter adapter(
@@ skipping 103 matching lines @@
 views::View* AutofillDialogViews::CreateDetailsContainer() {
   views::View* view = new views::View();
   // A box layout is used because it respects widget visibility.
   view->SetLayoutManager(
       new views::BoxLayout(views::BoxLayout::kVertical, 0, 0,
                            views::kRelatedControlVerticalSpacing));
 
   // Email.
   CreateDetailsSection(SECTION_EMAIL);
   view->AddChildView(email_.container);
+  // Credit card.
+  CreateDetailsSection(SECTION_CC);
+  view->AddChildView(cc_.container);
   // Billing.
-  CreateBillingSection();
+  CreateDetailsSection(SECTION_BILLING);
   view->AddChildView(billing_.container);
   // Shipping.
   CreateDetailsSection(SECTION_SHIPPING);
   view->AddChildView(shipping_.container);
-  shipping_.container->SetVisible(!use_billing_for_shipping_->checked());
 
   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);
 
   DetailsGroup* group = GroupForSection(section);
   // 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).
-  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,
                         views::GridLayout::LEADING,
                         1,
                         views::GridLayout::USE_PREF,
                         0,
                         0);
   column_set->AddColumn(views::GridLayout::CENTER,
                         views::GridLayout::LEADING,
                         0,
                         views::GridLayout::USE_PREF,
                         0,
                         0);
   layout->StartRow(0, kColumnSetId);
 
   // The |info_view| holds |manual_inputs| and |suggested_info|, allowing the
   // dialog toggle which is shown.
   views::View* info_view = new views::View();
   info_view->SetLayoutManager(
       new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0));
+
+  if (section == SECTION_SHIPPING) {
+    use_billing_for_shipping_ =
+        new views::Checkbox(controller_->UseBillingForShippingText());
+    use_billing_for_shipping_->SetChecked(true);
+    use_billing_for_shipping_->set_listener(this);
+    info_view->AddChildView(use_billing_for_shipping_);
+  }
+
   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::ImageButton* menu_button = new views::ImageButton(this);
   ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
@@ skipping 72 matching lines @@
     }
   }
 
   return view;
 }
 
 void AutofillDialogViews::UpdateDetailsGroupState(const DetailsGroup& group) {
   string16 suggestion_text =
       controller_->SuggestionTextForSection(group.section);
   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.section == SECTION_SHIPPING) {
+    bool show_checkbox = !show_suggestions;
+    // When the checkbox is going from hidden to visible, it's because the
+    // user clicked "Enter new address". Reset the checkbox to unchecked in this
+    // case.
+    if (show_checkbox && !use_billing_for_shipping_->visible())
+      use_billing_for_shipping_->SetChecked(false);
+
+    use_billing_for_shipping_->SetVisible(show_checkbox);
+    group.manual_input->SetVisible(
+        show_checkbox && !use_billing_for_shipping_->checked());
+  } else {
+    group.manual_input->SetVisible(!show_suggestions);
+  }
+
   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) {
@@ skipping 14 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