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

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

Issue 2872033007: [Payments] Do not show error on initial load of address screen (Closed)
Patch Set: deps check Created 3 years, 7 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
« no previous file with comments | « components/payments/content/BUILD.gn ('k') | components/payments/content/payment_request_spec.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/payments/content/payment_request_spec.h
diff --git a/components/payments/content/payment_request_spec.h b/components/payments/content/payment_request_spec.h
index 98e7d6733a7024df36c296113fc1cb14932fdc24..e7bc5ff2b285eb460dfa5f45de583b65bb194e02 100644
--- a/components/payments/content/payment_request_spec.h
+++ b/components/payments/content/payment_request_spec.h
@@ -121,10 +121,13 @@ class PaymentRequestSpec : public PaymentOptionsProvider {
void PopulateValidatedMethodData(
const std::vector<mojom::PaymentMethodDataPtr>& method_data);
- // Updates the selected_shipping_option based on the data passed to this
+ // Updates the |selected_shipping_option| based on the data passed to this
// payment request by the website. This will set selected_shipping_option_ to
- // the last option marked selected in the options array.
- void UpdateSelectedShippingOption();
+ // the last option marked selected in the options array. If no options are
+ // provided and this method is called |after_update|, it means the merchant
+ // doesn't ship to this location. In this case,
+ // |selected_shipping_option_error_| will be set.
+ void UpdateSelectedShippingOption(bool after_update);
// Will notify all observers that the spec has changed.
void NotifyOnSpecUpdated();
« no previous file with comments | « components/payments/content/BUILD.gn ('k') | components/payments/content/payment_request_spec.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698