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

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

Issue 2552693002: [Merge M-56] Rename canMakeActivePayment to canMakePayment (Closed)
Patch Set: Created 4 years 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/PaymentRequest.cpp
diff --git a/third_party/WebKit/Source/modules/payments/PaymentRequest.cpp b/third_party/WebKit/Source/modules/payments/PaymentRequest.cpp
index 657b8590fc0bc3c6e90c35ce32a05f734164eeaf..b91d35e1884df546c481d4c78320ee738d1ecfef 100644
--- a/third_party/WebKit/Source/modules/payments/PaymentRequest.cpp
+++ b/third_party/WebKit/Source/modules/payments/PaymentRequest.cpp
@@ -37,7 +37,7 @@
#include "wtf/HashSet.h"
#include <utility>
-using payments::mojom::blink::ActivePaymentQueryResult;
+using payments::mojom::blink::CanMakePaymentQueryResult;
using payments::mojom::blink::PaymentAddressPtr;
using payments::mojom::blink::PaymentCurrencyAmount;
using payments::mojom::blink::PaymentCurrencyAmountPtr;
@@ -594,18 +594,18 @@ ScriptPromise PaymentRequest::abort(ScriptState* scriptState) {
return m_abortResolver->promise();
}
-ScriptPromise PaymentRequest::canMakeActivePayment(ScriptState* scriptState) {
- if (!m_paymentProvider.is_bound() || m_canMakeActivePaymentResolver ||
+ScriptPromise PaymentRequest::canMakePayment(ScriptState* scriptState) {
+ if (!m_paymentProvider.is_bound() || m_canMakePaymentResolver ||
!scriptState->contextIsValid()) {
return ScriptPromise::rejectWithDOMException(
scriptState, DOMException::create(InvalidStateError,
"Cannot query payment request"));
}
- m_paymentProvider->CanMakeActivePayment();
+ m_paymentProvider->CanMakePayment();
- m_canMakeActivePaymentResolver = ScriptPromiseResolver::create(scriptState);
- return m_canMakeActivePaymentResolver->promise();
+ m_canMakePaymentResolver = ScriptPromiseResolver::create(scriptState);
+ return m_canMakePaymentResolver->promise();
}
bool PaymentRequest::hasPendingActivity() const {
@@ -700,7 +700,7 @@ DEFINE_TRACE(PaymentRequest) {
visitor->trace(m_showResolver);
visitor->trace(m_completeResolver);
visitor->trace(m_abortResolver);
- visitor->trace(m_canMakeActivePaymentResolver);
+ visitor->trace(m_canMakePaymentResolver);
EventTargetWithInlineData::trace(visitor);
ContextLifecycleObserver::trace(visitor);
}
@@ -897,8 +897,8 @@ void PaymentRequest::OnError(PaymentErrorReason error) {
if (m_abortResolver)
m_abortResolver->reject(DOMException::create(ec, message));
- if (m_canMakeActivePaymentResolver)
- m_canMakeActivePaymentResolver->reject(DOMException::create(ec, message));
+ if (m_canMakePaymentResolver)
+ m_canMakePaymentResolver->reject(DOMException::create(ec, message));
} else {
if (m_completeResolver)
m_completeResolver->reject(message);
@@ -909,8 +909,8 @@ void PaymentRequest::OnError(PaymentErrorReason error) {
if (m_abortResolver)
m_abortResolver->reject(message);
- if (m_canMakeActivePaymentResolver)
- m_canMakeActivePaymentResolver->reject(message);
+ if (m_canMakePaymentResolver)
+ m_canMakePaymentResolver->reject(message);
}
clearResolversAndCloseMojoConnection();
@@ -937,23 +937,23 @@ void PaymentRequest::OnAbort(bool abortedSuccessfully) {
clearResolversAndCloseMojoConnection();
}
-void PaymentRequest::OnCanMakeActivePayment(ActivePaymentQueryResult result) {
- DCHECK(m_canMakeActivePaymentResolver);
+void PaymentRequest::OnCanMakePayment(CanMakePaymentQueryResult result) {
+ DCHECK(m_canMakePaymentResolver);
switch (result) {
- case ActivePaymentQueryResult::CAN_MAKE_ACTIVE_PAYMENT:
- m_canMakeActivePaymentResolver->resolve(true);
+ case CanMakePaymentQueryResult::CAN_MAKE_PAYMENT:
+ m_canMakePaymentResolver->resolve(true);
break;
- case ActivePaymentQueryResult::CANNOT_MAKE_ACTIVE_PAYMENT:
- m_canMakeActivePaymentResolver->resolve(false);
+ case CanMakePaymentQueryResult::CANNOT_MAKE_PAYMENT:
+ m_canMakePaymentResolver->resolve(false);
break;
- case ActivePaymentQueryResult::QUERY_QUOTA_EXCEEDED:
- m_canMakeActivePaymentResolver->reject(
+ case CanMakePaymentQueryResult::QUERY_QUOTA_EXCEEDED:
+ m_canMakePaymentResolver->reject(
DOMException::create(QuotaExceededError, "Query quota exceeded"));
break;
}
- m_canMakeActivePaymentResolver.clear();
+ m_canMakePaymentResolver.clear();
}
void PaymentRequest::onCompleteTimeout(TimerBase*) {
@@ -966,7 +966,7 @@ void PaymentRequest::clearResolversAndCloseMojoConnection() {
m_completeResolver.clear();
m_showResolver.clear();
m_abortResolver.clear();
- m_canMakeActivePaymentResolver.clear();
+ m_canMakePaymentResolver.clear();
if (m_clientBinding.is_bound())
m_clientBinding.Close();
m_paymentProvider.reset();

Powered by Google App Engine
This is Rietveld 408576698