Title: [214179] trunk/Source/WebKit2
Revision
214179
Author
ander...@apple.com
Date
2017-03-20 11:30:19 -0700 (Mon, 20 Mar 2017)

Log Message

Fix a paste-o in WebPaymentCoordinatorProxy::platformCompletePaymentSession
https://bugs.webkit.org/show_bug.cgi?id=169881
rdar://problem/31030944

Reviewed by Dan Bernstein.

Don't always pass PKPaymentAuthorizationStatusFailure to the completion handler.

* UIProcess/ApplePay/cocoa/WebPaymentCoordinatorProxyCocoa.mm:
(WebKit::WebPaymentCoordinatorProxy::platformCompletePaymentSession):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (214178 => 214179)


--- trunk/Source/WebKit2/ChangeLog	2017-03-20 17:55:20 UTC (rev 214178)
+++ trunk/Source/WebKit2/ChangeLog	2017-03-20 18:30:19 UTC (rev 214179)
@@ -1,3 +1,16 @@
+2017-03-20  Anders Carlsson  <ander...@apple.com>
+
+        Fix a paste-o in WebPaymentCoordinatorProxy::platformCompletePaymentSession
+        https://bugs.webkit.org/show_bug.cgi?id=169881
+        rdar://problem/31030944
+
+        Reviewed by Dan Bernstein.
+
+        Don't always pass PKPaymentAuthorizationStatusFailure to the completion handler.
+
+        * UIProcess/ApplePay/cocoa/WebPaymentCoordinatorProxyCocoa.mm:
+        (WebKit::WebPaymentCoordinatorProxy::platformCompletePaymentSession):
+
 2017-03-20  Wenson Hsieh  <wenson_hs...@apple.com>
 
         TextIndicator should support a mode where selection rects are used to size the snapshot

Modified: trunk/Source/WebKit2/UIProcess/ApplePay/cocoa/WebPaymentCoordinatorProxyCocoa.mm (214178 => 214179)


--- trunk/Source/WebKit2/UIProcess/ApplePay/cocoa/WebPaymentCoordinatorProxyCocoa.mm	2017-03-20 17:55:20 UTC (rev 214178)
+++ trunk/Source/WebKit2/UIProcess/ApplePay/cocoa/WebPaymentCoordinatorProxyCocoa.mm	2017-03-20 18:30:19 UTC (rev 214179)
@@ -631,7 +631,7 @@
     m_paymentAuthorizationViewControllerDelegate->_didReachFinalState = WebCore::isFinalStateStatus(status);
 
 #if (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 101300) || (PLATFORM(IOS) && __IPHONE_OS_VERSION_MIN_REQUIRED >= 110000)
-    auto pkPaymentAuthorizationResult = adoptNS([allocPKPaymentAuthorizationResultInstance() initWithStatus:PKPaymentAuthorizationStatusFailure errors:result ? toNSErrors(result->errors).get() : @[ ]]);
+    auto pkPaymentAuthorizationResult = adoptNS([allocPKPaymentAuthorizationResultInstance() initWithStatus:toPKPaymentAuthorizationStatus(status) errors:result ? toNSErrors(result->errors).get() : @[ ]]);
     m_paymentAuthorizationViewControllerDelegate->_paymentAuthorizedCompletion(pkPaymentAuthorizationResult.get());
 #else
     m_paymentAuthorizationViewControllerDelegate->_paymentAuthorizedCompletion(toPKPaymentAuthorizationStatus(status));
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to