[Apple Pay] Add new ApplePayInstallmentConfiguration members
rdar://problem/60703650
Created attachment 400019 [details] Patch
Created attachment 400022 [details] Patch
Comment on attachment 400022 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=400022&action=review > Source/WebCore/Modules/applepay/PaymentInstallmentConfiguration.mm:217 > + applicationMetadataDictionary = dynamic_objc_cast<NSDictionary>([NSJSONSerialization JSONObjectWithData:applicationMetadata options:0 error:nil]); Maybe we should pass an NSError** and return an exception here?
Comment on attachment 400022 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=400022&action=review >> Source/WebCore/Modules/applepay/PaymentInstallmentConfiguration.mm:217 >> + applicationMetadataDictionary = dynamic_objc_cast<NSDictionary>([NSJSONSerialization JSONObjectWithData:applicationMetadata options:0 error:nil]); > > Maybe we should pass an NSError** and return an exception here? -JSONObjectWithData:options:error: returns nil when an error occurs, so we will already return an exception in that case (as well as when the call succeeds but the JSON root value isn’t an object). Passing an NSError would let us include more diagnostic information in the Exception message, but I’m not sure we want to expose Foundation's error messages on the web.
Committed r262069: <https://trac.webkit.org/changeset/262069> All reviewed patches have been landed. Closing bug and clearing flags on attachment 400022 [details].
Fixed the build in r262072: <https://trac.webkit.org/changeset/262072>