[w3c/payment-handler] Editorial tweaks to " MethodData Population Algorithm" (#264)

> 3. Create a new empty Sequence.
> 4. Set dataList to the newly created Sequence. 

These should just be a single step.

> 5. For each item in PaymentRequest@[[methodData]] in the corresponding payment request, perform the following steps:
>    1. Set inData to the item under consideration. 

Just use "FOr each _inData_ in ... well...

This algorithm needs to take a _paymentRequest_ variable as an argument. Then do _paymentRequest_.[[methodData]].

> skip the remaining substeps and move on to the next item (if any). 

You can just say "[continue](https://infra.spec.whatwg.org/#iteration-continue)".

> 4. Create a new PaymentMethodData object.
> 5. Set outData to the newly created PaymentMethodData. 

These can just be one step

> Set methodData to dataList. 

What is methodData? I think this algorithm needs to take a PaymentRequestEvent, and set _event_.methodData to _dataList_. Otherwise you're just setting a variable to a value, but that variable has no meaning outside this algorithm, so your entire algorithm did nothing.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/payment-handler/issues/264