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

components/payments/content/payment_request.h

 // Copyright 2016 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.
 
 #ifndef COMPONENTS_PAYMENTS_CONTENT_PAYMENT_REQUEST_H_
 #define COMPONENTS_PAYMENTS_CONTENT_PAYMENT_REQUEST_H_
 
 #include <memory>
 #include <vector>
 
@@ skipping 16 matching lines @@
 // This class manages the interaction between the renderer (through the
 // PaymentRequestClient and Mojo stub implementation) and the UI (through the
 // PaymentRequestDelegate). The API user (merchant) specification (supported
 // payment methods, required information, order details) is stored in
 // PaymentRequestSpec, and the current user selection state (and related data)
 // is stored in PaymentRequestSpec.
 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
   void Init(mojom::PaymentRequestClientPtr client,
             std::vector<mojom::PaymentMethodDataPtr> method_data,
             mojom::PaymentDetailsPtr details,
             mojom::PaymentOptionsPtr options) override;
   void Show() override;
   void UpdateWith(mojom::PaymentDetailsPtr details) override {}
   void Abort() override;
@@ skipping 29 matching lines @@
   content::WebContents* web_contents_;
   std::unique_ptr<PaymentRequestDelegate> delegate_;
   // |manager_| owns this PaymentRequest.
   PaymentRequestWebContentsManager* manager_;
   mojo::Binding<mojom::PaymentRequest> binding_;
   mojom::PaymentRequestClientPtr client_;
 
   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);
 };
 
 }  // namespace payments
 
 #endif  // COMPONENTS_PAYMENTS_CONTENT_PAYMENT_REQUEST_H_