[Payment Request] Resolve PaymentRequest.show()'s accept promise when a payment is authorized
Created attachment 324468 [details] Patch
Attachment 324468 [details] did not pass style-queue: ERROR: Source/WebCore/Modules/applepay/paymentrequest/ApplePayPaymentHandler.cpp:74: Code inside a namespace should not be indented. [whitespace/indent] [4] Total errors found: 1 in 41 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 324468 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=324468&action=review > Source/WebCore/Modules/applepay/paymentrequest/ApplePayPaymentHandler.cpp:287 > + auto address = PaymentAddress::create(); > + address->setCountry(contact.countryCode); Could we make a constructor that takes these values so if we add one later we won't forget to add setters in all the necessary places? > Source/WebCore/Modules/paymentrequest/PaymentAddress.h:38 > + static Ref<PaymentAddress> create() Yeah, let's add a meaningful constructor instead of a bunch of setters.
Created attachment 324576 [details] Patch
rdar://problem/33542813
Comment on attachment 324576 [details] Patch Rejecting attachment 324576 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 324576, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: file LayoutTests/http/tests/paymentrequest/payment-response-payerName-attribute.https.html patching file LayoutTests/http/tests/paymentrequest/payment-response-payerPhone-attribute.https-expected.txt patching file LayoutTests/http/tests/paymentrequest/payment-response-payerPhone-attribute.https.html patching file LayoutTests/http/tests/paymentrequest/resources/helpers.js Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/4960745
Created attachment 324582 [details] Patch
Comment on attachment 324582 [details] Patch Clearing flags on attachment: 324582 Committed r223855: <https://trac.webkit.org/changeset/223855>
All reviewed patches have been landed. Closing bug.