[Payment Request] Displays appropriate footer if first transaction completed

components/payments/core/BUILD.gn

Index: components/payments/core/BUILD.gn
diff --git a/components/payments/core/BUILD.gn b/components/payments/core/BUILD.gn
index c1a9b60fcd39aadd4e34147bc107898c4fa2a79c..4b9d266172bab83204a06eed4238e1e5890849b0 100644
--- a/components/payments/core/BUILD.gn
+++ b/components/payments/core/BUILD.gn
@@ -61,6 +61,8 @@ static_library("core") {
 static_library("test_support") {
   testonly = true
   sources = [
+    "payments_test_util.cc",

[Mathieu, 2017/07/05 13:26:28]

nit: payment_test_utils

[Moe, 2017/07/05 13:59:23]

Hmm :? I've received the opposite feedback in the past, which is to prefer
_util.h over _utils.h

[Mathieu, 2017/07/05 14:23:00]

From code search test_util does seem a little more prevalent, you're right
(perhaps a 60/40 split between the two). I was saying this because we have
autofill_test_utils and now having this as test_util feels weird. We should pick
one

[Eugene But (OOO till 7-30), 2017/07/05 14:47:31]

FWIW: _util is more prevalent than _utils

[Moe, 2017/07/05 16:01:46]

It makes sense to move toward what's more prevalent. Let's change the
autofill_test_utils to autofill_test_util in a separate CL.

+    "payments_test_util.h",
     "test_address_normalizer.cc",
     "test_address_normalizer.h",
     "test_payment_request_delegate.cc",
@@ -71,6 +73,9 @@ static_library("test_support") {
     ":core",
     "//base",
     "//components/autofill/core/browser",
+    "//components/autofill/core/browser:test_support",
+    "//components/pref_registry",
+    "//components/prefs",
   ]
 }