[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());
+}
+
+// views::CustomButton:
+void PaymentRequestRowView::StateChanged() {
+  if (state() == views::Button::STATE_HOVERED ||
+      state() == views::Button::STATE_PRESSED) {

[please use gerrit instead, 2017/01/25 16:22:55]

nit: I'd prefer hover and press to have different border colors, but don't take
my UX advise as gospel. We have UX people for that ;-)

[anthonyvd, 2017/01/25 17:56:33]

Yeah the mocks aren't too specific about the difference between hover and
pressed state but I'd assume we'll want a slightly different highlight. Borders
disappear when the row is highlighted though (in this case both the highlight
and border are the same color so the effect is achieved).

I'll loop in UXers to take a look when this is in Canary, this way they'll be
able to try it out and advise!

+    set_background(views::Background::CreateSolidBackground(SK_ColorLTGRAY));
+  } else {
+    set_background(nullptr);
+  }
+}
+
+}  // namespace payments