[Web Payments] Refactor the row display code.

chrome/browser/ui/views/payments/payment_request_row_view.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_row_view.h"
+
+#include "chrome/browser/ui/views/payments/payment_request_views_util.h"
+#include "third_party/skia/include/core/SkColor.h"
+#include "ui/views/background.h"
+#include "ui/views/border.h"
+
+namespace payments {
+
+PaymentRequestRowView::PaymentRequestRowView(
+    views::ButtonListener* listener)
+  : views::CustomButton(listener) {
+  SetBorder(payments::CreatePaymentRequestRowBorder());
+}
+
+PaymentRequestRowView::~PaymentRequestRowView() {}
+
+// views::CustomButton:
+void PaymentRequestRowView::StateChanged() {
+  if (state() == views::Button::STATE_HOVERED ||
+      state() == views::Button::STATE_PRESSED) {
+    set_background(views::Background::CreateSolidBackground(SK_ColorLTGRAY));
+  } else {
+    set_background(nullptr);
+  }

[sky, 2017/01/25 21:45:20]

Don't you need a SchedulePaint here? View should really do that for you, but it
doesn't currently.

[anthonyvd, 2017/01/26 18:56:06]

I doesn't *look* like it, CustomButton calls it itself after calling
StateChanged() and that seems to be the only call to StateChanged ever.

See
https://cs.chromium.org/chromium/src/ui/views/controls/button/custom_button.c...

+}
+
+}  // namespace payments