PaymentRequest API.

third_party/WebKit/Source/modules/payments/PaymentResponse.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 PaymentResponse_h
+#define PaymentResponse_h
+
+#include "bindings/core/v8/ScriptPromise.h"
+#include "bindings/core/v8/ScriptValue.h"
+#include "bindings/core/v8/ScriptWrappable.h"
+#include "platform/heap/Handle.h"
+#include "wtf/Noncopyable.h"
+#include "wtf/text/WTFString.h"
+
+namespace blink {
+
+class ScriptState;
+
+class PaymentResponse final : public GarbageCollectedFinalized<PaymentResponse>, public ScriptWrappable {
+    DEFINE_WRAPPERTYPEINFO();
+    WTF_MAKE_NONCOPYABLE(PaymentResponse);
+
+public:
+    PaymentResponse();
+    virtual ~PaymentResponse();
+
+    const String& methodName() const { return m_methodName; }
+    const ScriptValue& details() const { return m_details; }

[Marijn Kruisselbrink, 2016/02/19 22:39:44]

You should probably get rid of m_details and return either a default constructed
ScriptValue or ScriptValue::createNull directly (and change the method to return
by value).

Or go all the way and change m_details to Vector<char> and return
ScriptValue(scriptState,
SerializedScriptValueFactory::createFromWireBytes()->deserialize(scriptState->isolate());

(similar to Notification::data(), with the exception that the caching that is
done in that getter is wrong and has the same issues around ScriptValue)

[please use gerrit instead, 2016/02/26 21:06:28]

Storing JSON as a string and parsing it into a ScriptValue.

+
+    ScriptPromise complete(ScriptState*, bool success);
+
+    DEFINE_INLINE_TRACE() {}
+
+private:
+    String m_methodName;
+    ScriptValue m_details;
+};
+
+} // namespace blink
+
+#endif // PaymentResponse_h