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

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

Issue 2193183002: Print proper debug message in case of change event timeout. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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 PaymentRequest_h 5 #ifndef PaymentRequest_h
6 #define PaymentRequest_h 6 #define PaymentRequest_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 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 61
62 // EventTargetWithInlineData: 62 // EventTargetWithInlineData:
63 const AtomicString& interfaceName() const override; 63 const AtomicString& interfaceName() const override;
64 ExecutionContext* getExecutionContext() const override; 64 ExecutionContext* getExecutionContext() const override;
65 65
66 // PaymentCompleter: 66 // PaymentCompleter:
67 ScriptPromise complete(ScriptState*, PaymentComplete result) override; 67 ScriptPromise complete(ScriptState*, PaymentComplete result) override;
68 68
69 // PaymentUpdater: 69 // PaymentUpdater:
70 void onUpdatePaymentDetails(const ScriptValue& detailsScriptValue) override; 70 void onUpdatePaymentDetails(const ScriptValue& detailsScriptValue) override;
71 void onUpdatePaymentDetailsFailure(const ScriptValue& error) override; 71 void onUpdatePaymentDetailsFailure(const ScriptValue& error) override;
please use gerrit instead 2016/07/29 15:40:50 Let's change this ScriptValue to String
pals 2016/08/01 11:18:23 Done.
72 void abortPaymentRequest(const String& message) override;
please use gerrit instead 2016/07/29 15:40:50 Let's remove this method
pals 2016/08/01 11:18:23 Done.
72 73
73 DECLARE_TRACE(); 74 DECLARE_TRACE();
74 75
75 private: 76 private:
76 PaymentRequest(ScriptState*, const HeapVector<PaymentMethodData>&, const Pay mentDetails&, const PaymentOptions&, ExceptionState&); 77 PaymentRequest(ScriptState*, const HeapVector<PaymentMethodData>&, const Pay mentDetails&, const PaymentOptions&, ExceptionState&);
77 78
78 // LifecycleObserver: 79 // LifecycleObserver:
79 void contextDestroyed() override; 80 void contextDestroyed() override;
80 81
81 // ActiveScriptWrappable: 82 // ActiveScriptWrappable:
(...skipping 18 matching lines...) Expand all
100 Member<ScriptPromiseResolver> m_showResolver; 101 Member<ScriptPromiseResolver> m_showResolver;
101 Member<ScriptPromiseResolver> m_completeResolver; 102 Member<ScriptPromiseResolver> m_completeResolver;
102 Member<ScriptPromiseResolver> m_abortResolver; 103 Member<ScriptPromiseResolver> m_abortResolver;
103 mojom::blink::PaymentRequestPtr m_paymentProvider; 104 mojom::blink::PaymentRequestPtr m_paymentProvider;
104 mojo::Binding<mojom::blink::PaymentRequestClient> m_clientBinding; 105 mojo::Binding<mojom::blink::PaymentRequestClient> m_clientBinding;
105 }; 106 };
106 107
107 } // namespace blink 108 } // namespace blink
108 109
109 #endif // PaymentRequest_h 110 #endif // PaymentRequest_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698