Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(56)

Side by Side Diff: components/payments/core/payment_instrument.h

Issue 2757523002: [Payments] Use PaymentInstrument/AutofillPaymentInstrument throughout. (Closed)
Patch Set: remove instrument cache Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_PAYMENTS_CORE_PAYMENT_INSTRUMENT_H_ 5 #ifndef COMPONENTS_PAYMENTS_CORE_PAYMENT_INSTRUMENT_H_
6 #define COMPONENTS_PAYMENTS_CORE_PAYMENT_INSTRUMENT_H_ 6 #define COMPONENTS_PAYMENTS_CORE_PAYMENT_INSTRUMENT_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 10
11 #include "base/macros.h" 11 #include "base/macros.h"
12 #include "base/strings/string16.h"
12 13
13 namespace payments { 14 namespace payments {
14 15
15 // Base class which represents a form of payment in Payment Request. 16 // Base class which represents a form of payment in Payment Request.
16 class PaymentInstrument { 17 class PaymentInstrument {
17 public: 18 public:
18 class Delegate { 19 class Delegate {
19 public: 20 public:
20 virtual ~Delegate() {} 21 virtual ~Delegate() {}
21 22
22 // Should be called with method name (e.g., "visa") and json-serialized 23 // Should be called with method name (e.g., "visa") and json-serialized
23 // stringified details. 24 // stringified details.
24 virtual void OnInstrumentDetailsReady( 25 virtual void OnInstrumentDetailsReady(
25 const std::string& method_name, 26 const std::string& method_name,
26 const std::string& stringified_details) = 0; 27 const std::string& stringified_details) = 0;
27 28
28 virtual void OnInstrumentDetailsError() = 0; 29 virtual void OnInstrumentDetailsError() = 0;
29 }; 30 };
30 31
31 virtual ~PaymentInstrument() {} 32 virtual ~PaymentInstrument();
32 33
33 // Will call into the |delegate| (can't be null) on success or error. 34 // Will call into the |delegate| (can't be null) on success or error.
34 virtual void InvokePaymentApp(Delegate* delegate) = 0; 35 virtual void InvokePaymentApp(Delegate* delegate) = 0;
36 virtual bool IsValid() = 0;
please use gerrit instead 2017/03/16 14:27:03 Add a comment.
Mathieu 2017/03/16 14:57:04 Done.
35 37
36 const std::string& method_name() { return method_name_; } 38 const std::string& method_name() { return method_name_; }
please use gerrit instead 2017/03/16 14:27:03 These methods should be const.
Mathieu 2017/03/16 14:57:04 Done.
39 const base::string16& label() { return label_; }
40 const base::string16& sublabel() { return sublabel_; }
41 int icon_resource_id() { return icon_resource_id_; }
37 42
38 protected: 43 protected:
39 explicit PaymentInstrument(const std::string& method_name) 44 PaymentInstrument(const std::string& method_name,
40 : method_name_(method_name) {} 45 const base::string16& label,
46 const base::string16& sublabel,
47 int icon_resource_id);
41 48
42 private: 49 private:
43 const std::string method_name_; 50 const std::string method_name_;
51 const base::string16 label_;
52 const base::string16 sublabel_;
53 int icon_resource_id_;
44 54
45 DISALLOW_COPY_AND_ASSIGN(PaymentInstrument); 55 DISALLOW_COPY_AND_ASSIGN(PaymentInstrument);
46 }; 56 };
47 57
48 } // namespace payments 58 } // namespace payments
49 59
50 #endif // COMPONENTS_PAYMENTS_CORE_PAYMENT_INSTRUMENT_H_ 60 #endif // COMPONENTS_PAYMENTS_CORE_PAYMENT_INSTRUMENT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698