tree 0ef23506a3c6966a2dd184182fa3f0057e6b1f45
parent a3b105722c6ee802944625a21900ab445639e179
author Rouslan Solomakhin <rouslan@chromium.org> 1668694250 +0000
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1668694250 +0000

[Web Payment] Reduce number of test delegates.

This patch merges a couple of test implementations of
ChromePaymentRequestDelegate into a single
TestChromePaymentRequestDelegate and removes its constructor
parameters, so it can be used more easily in the next patches.

This is step 1 in multi-step patch to make
ServiceWorkerPaymentAppFactory (instead of RenderFrameHost) own
ServiceWorkerPaymentAppFinder, in order to simplify object ownership in
Web Payment code. Steps:
1) Reduce # of test implementations of ChromePaymentRequestDelegate.
2) Move PaymentRequest constructor parameters into the delegate.
3) Tests affect ServiceWorkerPaymentAppFinder through the delegate.
4) ServiceWorkerPaymentAppFactory owns ServiceWorkerPaymentAppFinder.

Bug: 1382826
Change-Id: I9801deda8cb1d6e5cbee58b8165b398c270af5d0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4023844
Reviewed-by: Stephen McGruer <smcgruer@chromium.org>
Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1072770}