[Payments] Have an observer for canMakePayment, for testing.

components/payments/content/payment_request.h

Index: components/payments/content/payment_request.h
diff --git a/components/payments/content/payment_request.h b/components/payments/content/payment_request.h
index b9753dbb41a3d84ed52d7fe050c7ada9e8f33edc..dc229f0d9282469bdaa69e08c8b279b8bc11f8e1 100644
--- a/components/payments/content/payment_request.h
+++ b/components/payments/content/payment_request.h
@@ -34,10 +34,16 @@ class PaymentRequest : public mojom::PaymentRequest,
                        public PaymentRequestSpec::Observer,
                        public PaymentRequestState::Delegate {
  public:
+  class ObserverForTest {
+   public:
+    virtual void OnCanMakePaymentCalled() = 0;

[please use gerrit instead, 2017/03/27 18:12:32]

Need a virtual empty destructor.

[Mathieu, 2017/03/27 18:28:24]

Done.

+  };
+
   PaymentRequest(content::WebContents* web_contents,
                  std::unique_ptr<PaymentRequestDelegate> delegate,
                  PaymentRequestWebContentsManager* manager,
-                 mojo::InterfaceRequest<mojom::PaymentRequest> request);
+                 mojo::InterfaceRequest<mojom::PaymentRequest> request,
+                 ObserverForTest* observer_for_testing);
   ~PaymentRequest() override;
 
   // mojom::PaymentRequest
@@ -87,6 +93,9 @@ class PaymentRequest : public mojom::PaymentRequest,
   std::unique_ptr<PaymentRequestSpec> spec_;
   std::unique_ptr<PaymentRequestState> state_;
 
+  // May be null.

[please use gerrit instead, 2017/03/27 18:12:32]

, must outlive this object.

[Mathieu, 2017/03/27 18:28:24]

Done.

+  ObserverForTest* observer_for_testing_;
+
   DISALLOW_COPY_AND_ASSIGN(PaymentRequest);
 };