Give AddressInit's members defaults

marcoscaceres commented 5 years ago

Closes #853

Based on https://github.com/web-platform-tests/wpt/pull/14271

This moves what was in the prose (setting members to empty string) to the AddressInit IDL. That clarifies some ambiguities around null and undefined - as now these are always the empty string.

The following tasks have been completed:

[x] Confirmed there are no ReSpec errors/warnings.
[x] Modified Web platform tests - tested already.
[x] Modified MDN Docs - doesn't apply.
[x] Has undergone security/privacy review - refactor, doesn't apply

Implementation commitment:

[ ] Safari (link to issue)
[ ] Chrome
[ ] Firefox
[ ] Edge (public signal)

Optional, impact on Payment Handler spec?

Preview | Diff

marcoscaceres commented 5 years ago

Need implementations to update their IDL bindings, but this could land as is.

marcoscaceres commented 5 years ago

@danyao, @zouhir, or @aestes, would you be supportive of this change? If so, would appriciate a link to a relevant bug 🙏

danyao commented 5 years ago

Filed https://bugs.chromium.org/p/chromium/issues/detail?id=959393

marcoscaceres commented 4 years ago

Merging based on Chrome and Firefox willing to implement the defaults.

w3c / payment-request

Give AddressInit's members defaults #852