[Web Payments] Add "pencil" edit button to lists.

chrome/browser/ui/views/payments/payment_request_item_list.cc

 // Copyright 2017 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/payments/payment_request_item_list.h"
 
 #include "chrome/browser/ui/views/payments/payment_request_dialog_view_ids.h"
 #include "chrome/browser/ui/views/payments/payment_request_row_view.h"
 #include "chrome/browser/ui/views/payments/payment_request_views_util.h"
 #include "components/payments/content/payment_request_state.h"
 #include "ui/gfx/paint_vector_icon.h"
+#include "ui/vector_icons/vector_icons.h"
+#include "ui/views/controls/button/image_button.h"
+#include "ui/views/controls/button/image_button_factory.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/grid_layout.h"
 #include "ui/views/vector_icons.h"
 #include "ui/views/view.h"
 
 namespace payments {
 
 namespace {
 
 const SkColor kCheckmarkColor = 0xFF609265;
 
 }  // namespace
 
 PaymentRequestItemList::Item::Item(PaymentRequestSpec* spec,
                                    PaymentRequestState* state,
                                    PaymentRequestItemList* list,
-                                   bool selected)
-    : spec_(spec), state_(state), list_(list), selected_(selected) {}
+                                   bool selected,
+                                   bool show_edit_button)
+    : spec_(spec), state_(state), list_(list), selected_(selected) {
+  if (show_edit_button) {
+    edit_button_.reset(views::CreateVectorImageButton(this));
+    edit_button_->set_owned_by_client();
+    views::SetImageFromVectorIcon(edit_button_.get(), ui::kEditPenIcon);
+    constexpr int kBackArrowSize = 16;

[Mathieu, 2017/05/08 14:08:02]

fix

[anthonyvd, 2017/05/08 14:45:54]

Done.

+    edit_button_->SetSize(gfx::Size(kBackArrowSize, kBackArrowSize));
+    edit_button_->SetFocusBehavior(views::View::FocusBehavior::ALWAYS);
+    edit_button_->set_id(static_cast<int>(DialogViewID::EDIT_ITEM_BUTTON));
+  }
+}
 
 PaymentRequestItemList::Item::~Item() {}
 
 views::View* PaymentRequestItemList::Item::GetItemView() {
   if (!item_view_) {
     item_view_ = CreateItemView();
     item_view_->set_owned_by_client();
   }
 
   return item_view_.get();
@@ skipping 46 matching lines @@
   views::ColumnSet* columns = layout->AddColumnSet(0);
   columns->AddColumn(views::GridLayout::FILL, views::GridLayout::LEADING, 1,
                      views::GridLayout::USE_PREF, 0, 0);
 
   columns->AddPaddingColumn(1, 0);
 
   // Add a column for the checkmark shown next to the selected profile.
   columns->AddColumn(views::GridLayout::TRAILING, views::GridLayout::CENTER, 0,
                      views::GridLayout::USE_PREF, 0, 0);
 
-  // Add a column for the extra_view, which comes after the checkmark.
-  columns->AddColumn(views::GridLayout::TRAILING, views::GridLayout::CENTER, 0,
-                     views::GridLayout::USE_PREF, 0, 0);
+  constexpr int kExtraViewSpacing = 10;
+  std::unique_ptr<views::View> extra_view = CreateExtraView();
+  if (extra_view) {
+    columns->AddPaddingColumn(0, kExtraViewSpacing);
+    // Add a column for the extra_view, which comes after the checkmark.
+    columns->AddColumn(views::GridLayout::TRAILING, views::GridLayout::CENTER,
+                       0, views::GridLayout::USE_PREF, 0, 0);
+  }
+

[Mathieu, 2017/05/08 14:08:02]

I would find it easier to reason if we had CreateEditButton() called here? Why
do it in the constructor?

[anthonyvd, 2017/05/08 14:45:54]

This way avoided having to store the show_edit_button bool. Changed it since
it's probably better to delay creating the button anyways.

+  if (edit_button_) {
+    columns->AddPaddingColumn(0, kExtraViewSpacing);
+    // Add a column for the edit_button if it exists.
+    columns->AddColumn(views::GridLayout::TRAILING, views::GridLayout::CENTER,
+                       0, views::GridLayout::USE_PREF, 0, 0);
+  }
 
   layout->StartRow(0, 0);
   content->set_can_process_events_within_subtree(false);
   layout->AddView(content.release());
 
   checkmark_ = CreateCheckmark(selected());
   layout->AddView(checkmark_.get());
 
-  std::unique_ptr<views::View> extra_view = CreateExtraView();
   if (extra_view)
     layout->AddView(extra_view.release());
 
+  if (edit_button_)
+    layout->AddView(edit_button_.get());
+
   return std::move(row);
 }
 
 void PaymentRequestItemList::Item::ButtonPressed(views::Button* sender,
                                                  const ui::Event& event) {
-  if (CanBeSelected()) {
+  if (sender == edit_button_.get()) {
+    EditButtonPressed();
+  } else if (CanBeSelected()) {
     list()->SelectItem(this);
   } else {
     PerformSelectionFallback();
   }
 }
 
 PaymentRequestItemList::PaymentRequestItemList() : selected_item_(nullptr) {}
 
 PaymentRequestItemList::~PaymentRequestItemList() {}
 
@@ skipping 37 matching lines @@
   // It's possible that no item is currently selected, either during list
   // creation or in the middle of the selection operation when the previously
   // selected item has been deselected but the new one isn't selected yet.
   if (selected_item_)
     selected_item_->SetSelected(/*selected=*/false, /*notify=*/true);
 
   selected_item_ = nullptr;
 }
 
 }  // namespace payments