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

Unified Diff: third_party/WebKit/Source/modules/payments/PaymentRequestDetailsTest.cpp

Issue 2697123003: Use ExecutionContext instead of Document in PaymentRequest constructor. (Closed)
Patch Set: ExecutionContext in constructor only Created 3 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/payments/PaymentRequestDetailsTest.cpp
diff --git a/third_party/WebKit/Source/modules/payments/PaymentRequestDetailsTest.cpp b/third_party/WebKit/Source/modules/payments/PaymentRequestDetailsTest.cpp
index dc0b5694e88ffe229d4603f71f7ee4c7b5d3db23..13ba1bc4f05d69ce6b980a92d60975f1f7e01c21 100644
--- a/third_party/WebKit/Source/modules/payments/PaymentRequestDetailsTest.cpp
+++ b/third_party/WebKit/Source/modules/payments/PaymentRequestDetailsTest.cpp
@@ -4,6 +4,7 @@
#include "modules/payments/PaymentRequest.h"
+#include <ostream> // NOLINT
#include "bindings/core/v8/ExceptionState.h"
#include "bindings/core/v8/ScriptState.h"
#include "bindings/core/v8/V8BindingForTesting.h"
@@ -13,7 +14,6 @@
#include "modules/payments/PaymentOptions.h"
#include "modules/payments/PaymentTestHelper.h"
#include "testing/gtest/include/gtest/gtest.h"
-#include <ostream> // NOLINT
namespace blink {
namespace {
@@ -132,9 +132,9 @@ TEST_P(PaymentRequestDetailsTest, ValidatesDetails) {
SecurityOrigin::create(KURL(KURL(), "https://www.example.com/")));
PaymentOptions options;
options.setRequestShipping(true);
- PaymentRequest::create(scope.document(), buildPaymentMethodDataForTest(),
- GetParam().buildDetails(), options,
- scope.getExceptionState());
+ PaymentRequest::create(
+ scope.getExecutionContext(), buildPaymentMethodDataForTest(),
+ GetParam().buildDetails(), options, scope.getExceptionState());
EXPECT_EQ(GetParam().expectException(),
scope.getExceptionState().hadException());

Powered by Google App Engine
This is Rietveld 408576698