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

Unified Diff: components/payments/content/payment_response_helper.h

Issue 2832003003: Revert of [Payments] Normalize Shipping Address sent to merchant on Desktop. (Closed)
Patch Set: Created 3 years, 8 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: components/payments/content/payment_response_helper.h
diff --git a/components/payments/content/payment_response_helper.h b/components/payments/content/payment_response_helper.h
index fde039d74482f36e8249521581b622815f006a8d..69e0383799e3128f9e27f44bc68505f6e6cd37b6 100644
--- a/components/payments/content/payment_response_helper.h
+++ b/components/payments/content/payment_response_helper.h
@@ -6,21 +6,21 @@
#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"
#include "components/payments/core/payment_instrument.h"
#include "components/payments/mojom/payment_request.mojom.h"
+namespace autofill {
+class AutofillProfile;
+} // namespace autofill
+
namespace payments {
-class PaymentRequestDelegate;
class PaymentRequestSpec;
// TODO(sebsg): Asynchronously normalize the billing and shipping addresses
// before adding them to the PaymentResponse.
// A helper class to facilitate the creation of the PaymentResponse.
-class PaymentResponseHelper : public PaymentInstrument::Delegate,
- AddressNormalizer::Delegate {
+class PaymentResponseHelper : public PaymentInstrument::Delegate {
public:
class Delegate {
public:
@@ -34,7 +34,6 @@
PaymentResponseHelper(const std::string& app_locale,
PaymentRequestSpec* spec,
PaymentInstrument* selected_instrument,
- PaymentRequestDelegate* payment_request_delegate,
autofill::AutofillProfile* selected_shipping_profile,
autofill::AutofillProfile* selected_contact_profile,
Delegate* delegate);
@@ -52,33 +51,17 @@
const std::string& stringified_details) override;
void OnInstrumentDetailsError() override {}
- // AddressNormalizer::Delegate
- void OnAddressNormalized(
- const autofill::AutofillProfile& normalized_profile) override;
- void OnCouldNotNormalize(const autofill::AutofillProfile& profile) override;
-
private:
- // Generates the Payment Response and sends it to the delegate.
- void GeneratePaymentResponse();
-
const std::string& app_locale_;
- bool is_waiting_for_shipping_address_normalization_;
- bool is_waiting_for_instrument_details_;
// 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_shipping_profile_;
autofill::AutofillProfile* selected_contact_profile_;
-
- autofill::AutofillProfile shipping_address_;
-
- // Instrument Details.
- std::string method_name_;
- std::string stringified_details_;
DISALLOW_COPY_AND_ASSIGN(PaymentResponseHelper);
};
« no previous file with comments | « components/payments/content/payment_request_state.cc ('k') | components/payments/content/payment_response_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698