[Payments] Normalize Shipping Address sent to merchant on Desktop.

chrome/browser/payments/chrome_payment_request_delegate.h

Index: chrome/browser/payments/chrome_payment_request_delegate.h
diff --git a/chrome/browser/payments/chrome_payment_request_delegate.h b/chrome/browser/payments/chrome_payment_request_delegate.h
index 69a9a2bbac899007e362dd1d82cc396f51e2490a..af1e2299d1a03c2a5f54d3ff8160fd3e4c931db1 100644
--- a/chrome/browser/payments/chrome_payment_request_delegate.h
+++ b/chrome/browser/payments/chrome_payment_request_delegate.h
@@ -22,7 +22,7 @@ class PaymentRequestDialog;
 class ChromePaymentRequestDelegate : public PaymentRequestDelegate {
  public:
   explicit ChromePaymentRequestDelegate(content::WebContents* web_contents);
-  ~ChromePaymentRequestDelegate() override {}
+  ~ChromePaymentRequestDelegate() override;
 
   // PaymentRequestDelegate:
   void ShowDialog(PaymentRequest* request) override;
@@ -37,10 +37,11 @@ class ChromePaymentRequestDelegate : public PaymentRequestDelegate {
       const autofill::CreditCard& credit_card,
       base::WeakPtr<autofill::payments::FullCardRequest::ResultDelegate>
           result_delegate) override;
-  std::unique_ptr<const ::i18n::addressinput::Source> GetAddressInputSource()
+  std::unique_ptr<::i18n::addressinput::Source> GetAddressInputSource()
       override;
   std::unique_ptr<::i18n::addressinput::Storage> GetAddressInputStorage()
       override;
+  AddressNormalizer* GetAddressNormalizer() override;
 
  protected:
   // Reference to the dialog so that we can satisfy calls to CloseDialog(). This
@@ -52,6 +53,9 @@ class ChromePaymentRequestDelegate : public PaymentRequestDelegate {
   // Not owned but outlives the PaymentRequest object that owns this.
   content::WebContents* web_contents_;
 
+  // The address normalizer to use for the duration of the Payment Request.
+  std::unique_ptr<AddressNormalizer> address_normalizer_;

[please use gerrit instead, 2017/04/20 20:15:20]

No point of using std::unique_ptr if the variable is initialized in the
constructor and never reset. This works just well:

AddressNormalizer address_normalizer_;

[sebsg, 2017/04/20 21:06:19]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(ChromePaymentRequestDelegate);
 };