Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(719)

Unified Diff: third_party/WebKit/Source/modules/payments/PaymentAddress.idl

Issue 2039863002: PaymentRequest: Sync up PaymentAddress with the payment request spec. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/payments/PaymentAddress.idl
diff --git a/third_party/WebKit/Source/modules/payments/PaymentAddress.idl b/third_party/WebKit/Source/modules/payments/PaymentAddress.idl
index cf9decab6d913b56729f87b0962cbe9d2356275d..83eff3247d3f5047804b093f408b33a0cefafbfd 100644
--- a/third_party/WebKit/Source/modules/payments/PaymentAddress.idl
+++ b/third_party/WebKit/Source/modules/payments/PaymentAddress.idl
@@ -7,14 +7,19 @@
[
RuntimeEnabled=PaymentRequest,
] interface PaymentAddress {
- readonly attribute DOMString regionCode;
- readonly attribute sequence<DOMString> addressLine;
- readonly attribute DOMString administrativeArea;
- readonly attribute DOMString locality;
+ readonly attribute DOMString country;
+ readonly attribute FrozenArray<DOMString> addressLine;
+ readonly attribute DOMString region;
+ readonly attribute DOMString city;
readonly attribute DOMString dependentLocality;
readonly attribute DOMString postalCode;
readonly attribute DOMString sortingCode;
readonly attribute DOMString languageCode;
readonly attribute DOMString organization;
readonly attribute DOMString recipient;
+ readonly attribute DOMString careOf;
please use gerrit instead 2016/06/06 16:01:15 +phone
zino 2016/06/06 17:23:16 Done.
+
+ // TODO(zino): Should add the following attribute as well.
+ // Please see http://crbug.com/617195
+ // readonly attribute DOMString phone;
please use gerrit instead 2016/06/06 16:01:15 Yes, let's add it here. http://crbug.com/617195 is
};

Powered by Google App Engine
This is Rietveld 408576698