[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),
+      show_edit_button_(show_edit_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);
+  // The space between the checkmark, extra view, and edit button.
+  constexpr int kExtraViewSpacing = 16;
+  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);
+  }
+
+  if (show_edit_button_) {
+    columns->AddPaddingColumn(0, kExtraViewSpacing);
+    // Add a column for the edit_button if it exists.
+    constexpr int kEditIconSize = 16;
+    columns->AddColumn(views::GridLayout::TRAILING, views::GridLayout::CENTER,
+                       0, views::GridLayout::FIXED, kEditIconSize,
+                       kEditIconSize);
+  }
 
   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 (show_edit_button_) {
+    views::ImageButton* edit_button = views::CreateVectorImageButton(this);
+    edit_button->set_owned_by_client();
+    views::SetImageFromVectorIcon(edit_button, ui::kEditIcon);
+    edit_button->SetFocusBehavior(views::View::FocusBehavior::ALWAYS);
+    edit_button->set_id(static_cast<int>(DialogViewID::EDIT_ITEM_BUTTON));
+    layout->AddView(edit_button);
+  }
+
   return std::move(row);
 }
 
 void PaymentRequestItemList::Item::ButtonPressed(views::Button* sender,
                                                  const ui::Event& event) {
-  if (CanBeSelected()) {
+  if (sender->id() == static_cast<int>(DialogViewID::EDIT_ITEM_BUTTON)) {

[Evan Stade, 2017/05/12 18:48:07]

nit: are both of these casts necessary? (here and L139)

[anthonyvd, 2017/05/12 20:16:48]

Yeah, since DialogViewID is an enum class not casting results in 

error: invalid operands to binary expression ('int' and
'payments::DialogViewID')

[Evan Stade, 2017/05/15 17:14:55]

what error do you get for the other one? Because clang doesn't complain when I
set_id with an enum value.

[anthonyvd, 2017/05/15 17:27:40]

error: cannot initialize a parameter of type 'int' with an rvalue of type
'payments::DialogViewID'
Yeah, enum class conversion rules are stricter than plain enum's.

+    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