Bug 1435155 - Redact the shipping address as it is visible before the…

… promise from PaymentRequest.show() resolves successfully. r=baku

Spec PR: w3c/payment-request#654

MozReview-Commit-ID: 2AiKI7htRhk

UltraBlame original commit: fbc6dec319be8067e0d26055e9c93a5fca74d718

dom/payments/PaymentRequest.cpp

@@ -107,6 +107,10 @@ NS_IMPL_CYCLE_COLLECTION_TRAVERSE
 (
 mShippingAddress
 )
+NS_IMPL_CYCLE_COLLECTION_TRAVERSE
+(
+mFullShippingAddress
+)
 NS_IMPL_CYCLE_COLLECTION_TRAVERSE_END
 NS_IMPL_CYCLE_COLLECTION_UNLINK_BEGIN_INHERITED
 (
@@ -133,6 +137,10 @@ NS_IMPL_CYCLE_COLLECTION_UNLINK
 (
 mShippingAddress
 )
+NS_IMPL_CYCLE_COLLECTION_UNLINK
+(
+mFullShippingAddress
+)
 NS_IMPL_CYCLE_COLLECTION_UNLINK_END
 NS_INTERFACE_MAP_BEGIN_CYCLE_COLLECTION
 (
@@ -3775,6 +3783,17 @@ aRv
 return
 ;
 }
+mShippingAddress
+.
+swap
+(
+mFullShippingAddress
+)
+;
+mFullShippingAddress
+=
+nullptr
+;
 RefPtr
 <
 PaymentResponse
@@ -4533,6 +4552,12 @@ nsAString
 aPhone
 )
 {
+nsTArray
+<
+nsString
+>
+emptyArray
+;
 mShippingAddress
 =
 new
@@ -4542,6 +4567,33 @@ GetOwner
 (
 )
 aCountry
+emptyArray
+aRegion
+aCity
+aDependentLocality
+aPostalCode
+aSortingCode
+aLanguageCode
+EmptyString
+(
+)
+EmptyString
+(
+)
+EmptyString
+(
+)
+)
+;
+mFullShippingAddress
+=
+new
+PaymentAddress
+(
+GetOwner
+(
+)
+aCountry
 aAddressLine
 aRegion
 aCity

dom/payments/PaymentRequest.h

@@ -662,6 +662,12 @@ PaymentAddress
 >
 mShippingAddress
 ;
+RefPtr
+<
+PaymentAddress
+>
+mFullShippingAddress
+;
 nsString
 mShippingOption
 ;

dom/payments/test/ShowPaymentChromeScript.js

@@ -184,6 +184,8 @@ San
 Bruno
 "
 "
+Test
+locality
 "
 "
 94066
@@ -195,6 +197,8 @@ Bruno
 en
 "
 "
+Testing
+Org
 "
 "
 Bill