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

Unified Diff: chrome/browser/ui/views/payments/shipping_address_editor_view_controller.h

Issue 2881643002: Focus first invalid field of payment request editor (Closed)
Patch Set: Last final nit :-) 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
Index: chrome/browser/ui/views/payments/shipping_address_editor_view_controller.h
diff --git a/chrome/browser/ui/views/payments/shipping_address_editor_view_controller.h b/chrome/browser/ui/views/payments/shipping_address_editor_view_controller.h
index be51bee15a62319790d9c532e22384d7936d045b..d14485fb886d3d67f018b5127362a326f7ca4dda 100644
--- a/chrome/browser/ui/views/payments/shipping_address_editor_view_controller.h
+++ b/chrome/browser/ui/views/payments/shipping_address_editor_view_controller.h
@@ -71,12 +71,15 @@ class ShippingAddressEditorViewController : public EditorViewController {
~ShippingAddressValidationDelegate() override;
// ValidationDelegate:
- bool ValidateTextfield(views::Textfield* textfield) override;
- bool ValidateCombobox(views::Combobox* combobox) override;
+ bool IsValidTextfield(views::Textfield* textfield) override;
+ bool IsValidCombobox(views::Combobox* combobox) override;
+ bool TextfieldValueChanged(views::Textfield* textfield) override;
+ bool ComboboxValueChanged(views::Combobox* combobox) override;
void ComboboxModelChanged(views::Combobox* combobox) override;
private:
- bool ValidateValue(const base::string16& value);
+ bool ValidateValue(const base::string16& value,
+ base::string16* error_message);
EditorField field_;

Powered by Google App Engine
This is Rietveld 408576698