PaymentRequest.abort() should return a promise.

third_party/WebKit/Source/modules/payments/PaymentRequestTestBase.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 PaymentRequestTestBase_h
+#define PaymentRequestTestBase_h
+
+#include "bindings/core/v8/ExceptionState.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "wtf/OwnPtr.h"
+#include "wtf/text/WTFString.h"
+
+namespace blink {
+
+class DummyPageHolder;
+class ScriptState;
+
+// A base fixture for PaymentRequest tests.
+class PaymentRequestTestBase : public testing::Test {

[haraken, 2016/06/09 01:57:59]

Can we use ScriptStateForTesting?

[please use gerrit instead, 2016/06/09 22:36:14]

ScriptStateForTesting needs an isolate(), which I can get only from
DummyPageHolder. So replacing DummyPageHolder with ScriptStateForTesting seems
infeasible.

[haraken, 2016/06/10 00:15:12]

You can use v8::Isolate::GetCurrent().

Or if you really need a dummy page, you can add DummyPageHolder to
ScriptStateForTesting.

[please use gerrit instead, 2016/06/10 21:18:20]

After experimenting, turns out I need a document with a frame. PaymentRequest
calls frame->isMainFrame() and frame->getServiceRegstry(). It's simplest to use
DummyPageHolder. Placing DummyPageHolder in ScriptStateForTesting seems like an
overkill: most tests do not need this. I've placed it into
ScriptStateForTestingWithPage instead. What do you think?

+public:
+    PaymentRequestTestBase();
+    ~PaymentRequestTestBase() override;
+
+    ScriptState* getScriptState();
+    ExceptionState& getExceptionState();
+
+    void setSecurityOrigin(const String&);
+
+private:
+    OwnPtr<DummyPageHolder> m_page;
+    TrackExceptionState m_exceptionState;
+};
+
+} // namespace blink
+
+#endif // PaymentRequestTestBase_h