[Payments] Normalize shipping address change on Desktop

components/payments/content/payment_response_helper.h

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_PAYMENTS_CONTENT_PAYMENT_RESPONSE_HELPER_H_
 #define COMPONENTS_PAYMENTS_CONTENT_PAYMENT_RESPONSE_HELPER_H_
 
 #include "base/macros.h"
 #include "components/autofill/core/browser/autofill_profile.h"
 #include "components/payments/core/address_normalizer.h"
@@ skipping 25 matching lines @@
                         PaymentInstrument* selected_instrument,
                         PaymentRequestDelegate* payment_request_delegate,
                         autofill::AutofillProfile* selected_shipping_profile,
                         autofill::AutofillProfile* selected_contact_profile,
                         Delegate* delegate);
   ~PaymentResponseHelper() override;
 
   // Returns a new mojo PaymentAddress based on the specified
   // |profile| and |app_locale|.
   static mojom::PaymentAddressPtr GetMojomPaymentAddressFromAutofillProfile(
-      const autofill::AutofillProfile* const profile,
+      const autofill::AutofillProfile& profile,
       const std::string& app_locale);
 
   // PaymentInstrument::Delegate
   void OnInstrumentDetailsReady(
       const std::string& method_name,
       const std::string& stringified_details) override;
   void OnInstrumentDetailsError() override {}
 
   // AddressNormalizer::Delegate
   void OnAddressNormalized(
@@ skipping 10 matching lines @@
 
   // Not owned, cannot be null.
   PaymentRequestSpec* spec_;
   Delegate* delegate_;
   PaymentInstrument* selected_instrument_;
   PaymentRequestDelegate* payment_request_delegate_;
 
   // Not owned, can be null (dependent on the spec).
   autofill::AutofillProfile* selected_contact_profile_;
 
   autofill::AutofillProfile shipping_address_;

[Mathieu, 2017/04/26 13:09:49]

// A copy of the shipping address, which will be normalized before being
included in the PaymentResponse.

[sebsg, 2017/04/26 19:28:54]

Thanks, changed the comment a bit as this address is a normalized copy.

 
   // Instrument Details.
   std::string method_name_;
   std::string stringified_details_;
 
   DISALLOW_COPY_AND_ASSIGN(PaymentResponseHelper);
 };
 
 }  // namespace payments
 
 #endif  // COMPONENTS_PAYMENTS_CONTENT_PAYMENT_RESPONSE_HELPER_H_