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

Side by Side Diff: third_party/WebKit/Source/modules/payments/PaymentResponse.h

Issue 2770193003: Implement request id in PaymentDetailsInit (Closed)
Patch Set: Add comment in mojom 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 PaymentResponse_h 5 #ifndef PaymentResponse_h
6 #define PaymentResponse_h 6 #define PaymentResponse_h
7 7
8 #include "bindings/core/v8/ScriptPromise.h" 8 #include "bindings/core/v8/ScriptPromise.h"
9 #include "bindings/core/v8/ScriptValue.h" 9 #include "bindings/core/v8/ScriptValue.h"
10 #include "bindings/core/v8/ScriptWrappable.h" 10 #include "bindings/core/v8/ScriptWrappable.h"
(...skipping 12 matching lines...) Expand all
23 class ScriptState; 23 class ScriptState;
24 24
25 class MODULES_EXPORT PaymentResponse final 25 class MODULES_EXPORT PaymentResponse final
26 : public GarbageCollectedFinalized<PaymentResponse>, 26 : public GarbageCollectedFinalized<PaymentResponse>,
27 public ScriptWrappable { 27 public ScriptWrappable {
28 DEFINE_WRAPPERTYPEINFO(); 28 DEFINE_WRAPPERTYPEINFO();
29 WTF_MAKE_NONCOPYABLE(PaymentResponse); 29 WTF_MAKE_NONCOPYABLE(PaymentResponse);
30 30
31 public: 31 public:
32 PaymentResponse(payments::mojom::blink::PaymentResponsePtr, 32 PaymentResponse(payments::mojom::blink::PaymentResponsePtr,
33 PaymentCompleter*); 33 PaymentCompleter*,
34 const String& requestId = "");
please use gerrit instead 2017/03/27 21:16:05 = "" would only be used in tests. I prefer to avoi
34 virtual ~PaymentResponse(); 35 virtual ~PaymentResponse();
35 36
36 ScriptValue toJSONForBinding(ScriptState*) const; 37 ScriptValue toJSONForBinding(ScriptState*) const;
37 38
39 const String& requestId() const { return m_requestId; }
38 const String& methodName() const { return m_methodName; } 40 const String& methodName() const { return m_methodName; }
39 ScriptValue details(ScriptState*, ExceptionState&) const; 41 ScriptValue details(ScriptState*, ExceptionState&) const;
40 PaymentAddress* shippingAddress() const { return m_shippingAddress.get(); } 42 PaymentAddress* shippingAddress() const { return m_shippingAddress.get(); }
41 const String& shippingOption() const { return m_shippingOption; } 43 const String& shippingOption() const { return m_shippingOption; }
42 const String& payerName() const { return m_payerName; } 44 const String& payerName() const { return m_payerName; }
43 const String& payerEmail() const { return m_payerEmail; } 45 const String& payerEmail() const { return m_payerEmail; }
44 const String& payerPhone() const { return m_payerPhone; } 46 const String& payerPhone() const { return m_payerPhone; }
45 47
46 ScriptPromise complete(ScriptState*, const String& result = ""); 48 ScriptPromise complete(ScriptState*, const String& result = "");
47 49
48 DECLARE_TRACE(); 50 DECLARE_TRACE();
49 51
50 private: 52 private:
53 String m_requestId;
51 String m_methodName; 54 String m_methodName;
52 String m_stringifiedDetails; 55 String m_stringifiedDetails;
53 Member<PaymentAddress> m_shippingAddress; 56 Member<PaymentAddress> m_shippingAddress;
54 String m_shippingOption; 57 String m_shippingOption;
55 String m_payerName; 58 String m_payerName;
56 String m_payerEmail; 59 String m_payerEmail;
57 String m_payerPhone; 60 String m_payerPhone;
58 Member<PaymentCompleter> m_paymentCompleter; 61 Member<PaymentCompleter> m_paymentCompleter;
59 }; 62 };
60 63
61 } // namespace blink 64 } // namespace blink
62 65
63 #endif // PaymentResponse_h 66 #endif // PaymentResponse_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698