PaymentRequest: Introduce PaymentDetailsInit and PaymentDetailsUpdate.

third_party/WebKit/Source/modules/payments/PaymentRequest.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 PaymentRequest_h
 #define PaymentRequest_h
 
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptValue.h"
 #include "bindings/core/v8/ScriptWrappable.h"
 #include "components/payments/content/payment_request.mojom-blink.h"
 #include "core/dom/ContextLifecycleObserver.h"
 #include "core/events/EventTarget.h"
 #include "modules/ModulesExport.h"
 #include "modules/payments/PaymentCompleter.h"
-#include "modules/payments/PaymentDetails.h"
+#include "modules/payments/PaymentDetailsInit.h"

[please use gerrit instead, 2017/03/21 13:42:21]

Please forward declare this instead of including it in the header.

[zino, 2017/03/22 16:43:45]

Done.

+#include "modules/payments/PaymentDetailsUpdate.h"

[please use gerrit instead, 2017/03/21 13:42:21]

Please do not include this here. It's not used.

[zino, 2017/03/22 16:43:45]

Done.

 #include "modules/payments/PaymentMethodData.h"
 #include "modules/payments/PaymentOptions.h"
 #include "modules/payments/PaymentUpdater.h"
 #include "mojo/public/cpp/bindings/binding.h"
 #include "platform/Timer.h"
 #include "platform/heap/Handle.h"
 #include "wtf/Compiler.h"
 #include "wtf/Noncopyable.h"
 #include "wtf/RefPtr.h"
 #include "wtf/Vector.h"
@@ skipping 14 matching lines @@
       public PaymentUpdater,
       public ContextLifecycleObserver,
       public ActiveScriptWrappable<PaymentRequest> {
   DEFINE_WRAPPERTYPEINFO();
   USING_GARBAGE_COLLECTED_MIXIN(PaymentRequest)
   WTF_MAKE_NONCOPYABLE(PaymentRequest);
 
  public:
   static PaymentRequest* create(ExecutionContext*,
                                 const HeapVector<PaymentMethodData>&,
-                                const PaymentDetails&,
+                                const PaymentDetailsInit&,
                                 ExceptionState&);
   static PaymentRequest* create(ExecutionContext*,
                                 const HeapVector<PaymentMethodData>&,
-                                const PaymentDetails&,
+                                const PaymentDetailsInit&,
                                 const PaymentOptions&,
                                 ExceptionState&);
 
   virtual ~PaymentRequest();
 
   ScriptPromise show(ScriptState*);
   ScriptPromise abort(ScriptState*);
 
   PaymentAddress* getShippingAddress() const { return m_shippingAddress.get(); }
   const String& shippingOption() const { return m_shippingOption; }
@@ skipping 18 matching lines @@
   void onUpdatePaymentDetails(const ScriptValue& detailsScriptValue) override;
   void onUpdatePaymentDetailsFailure(const String& error) override;
 
   DECLARE_TRACE();
 
   void onCompleteTimeoutForTesting();
 
  private:
   PaymentRequest(ExecutionContext*,
                  const HeapVector<PaymentMethodData>&,
-                 const PaymentDetails&,
+                 const PaymentDetailsInit&,
                  const PaymentOptions&,
                  ExceptionState&);
 
   // LifecycleObserver:
   void contextDestroyed(ExecutionContext*) override;
 
   // payments::mojom::blink::PaymentRequestClient:
   void OnShippingAddressChange(
       payments::mojom::blink::PaymentAddressPtr) override;
   void OnShippingOptionChange(const String& shippingOptionId) override;
@@ skipping 18 matching lines @@
   Member<ScriptPromiseResolver> m_abortResolver;
   Member<ScriptPromiseResolver> m_canMakePaymentResolver;
   payments::mojom::blink::PaymentRequestPtr m_paymentProvider;
   mojo::Binding<payments::mojom::blink::PaymentRequestClient> m_clientBinding;
   TaskRunnerTimer<PaymentRequest> m_completeTimer;
 };
 
 }  // namespace blink
 
 #endif  // PaymentRequest_h