PaymentHandler: Initial implementation for PaymentInstruments.

third_party/WebKit/Source/modules/payments/PaymentInstruments.h

Index: third_party/WebKit/Source/modules/payments/PaymentInstruments.h
diff --git a/third_party/WebKit/Source/modules/payments/PaymentInstruments.h b/third_party/WebKit/Source/modules/payments/PaymentInstruments.h
new file mode 100644
index 0000000000000000000000000000000000000000..98d1d09ced6dd2707b909975002fddd96ae2481e
--- /dev/null
+++ b/third_party/WebKit/Source/modules/payments/PaymentInstruments.h
@@ -0,0 +1,38 @@
+// Copyright 2017 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 PaymentInstruments_h
+#define PaymentInstruments_h
+
+#include "bindings/core/v8/ScriptWrappable.h"
+#include "modules/ModulesExport.h"
+#include "platform/heap/Handle.h"
+
+namespace blink {
+
+class PaymentInstrument;
+class ScriptPromise;
+
+class MODULES_EXPORT PaymentInstruments final

[please use gerrit instead, 2017/03/28 16:33:29]

#include "modules/ModulesExport.h"

[zino, 2017/03/29 15:51:23]

The header file was already included :)

+    : public GarbageCollectedFinalized<PaymentInstruments>,

[please use gerrit instead, 2017/03/28 16:33:29]

#include "platform/heap/GarbageCollected.h"

[zino, 2017/03/29 15:51:23]

I think #include "platform/heap/Handle.h" is enough according to oilpan guide.
https://chromium.googlesource.com/chromium/src/+/master/third_party/WebKit/So...

+      public ScriptWrappable {
+  DEFINE_WRAPPERTYPEINFO();
+  WTF_MAKE_NONCOPYABLE(PaymentInstruments);

[please use gerrit instead, 2017/03/28 16:33:29]

#include "platform/wtf/Noncopyable.h"

[zino, 2017/03/29 15:51:23]

Done.

+
+  DECLARE_TRACE();

[please use gerrit instead, 2017/03/28 16:33:29]

I believe DECLARE_TRACE() should be public.

Also:

#include "platform/heap/Visitor.h"

[zino, 2017/03/29 15:51:23]

It's my mistake. Done!
#include "platform/heap/Handle.h" is enough.

+
+ public:
+  PaymentInstruments();
+
+  ScriptPromise deleteInstrument(const String& instrumentKey);
+  ScriptPromise get(const String& instrumentKey);
+  ScriptPromise keys();
+  ScriptPromise has(const String& instrumentKey);

[please use gerrit instead, 2017/03/28 16:33:29]

#include "platform/wtf/text/WTFString.h"

[zino, 2017/03/29 15:51:23]

Done.

+  ScriptPromise set(const String& instrumentKey,
+                    const PaymentInstrument& details);
+};
+
+}  // namespace blink
+
+#endif  // PaymentInstruments_h