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

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

Issue 2881643002: Focus first invalid field of payment request editor (Closed)
Patch Set: Rebase 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/credit_card_editor_view_controller.h
diff --git a/chrome/browser/ui/views/payments/credit_card_editor_view_controller.h b/chrome/browser/ui/views/payments/credit_card_editor_view_controller.h
index dd69630db1eb6e967761916d548328c6c5e36593..c8af3d38cb576f54114231bc2fb8cd09c8beb3f7 100644
--- a/chrome/browser/ui/views/payments/credit_card_editor_view_controller.h
+++ b/chrome/browser/ui/views/payments/credit_card_editor_view_controller.h
@@ -77,13 +77,15 @@ class CreditCardEditorViewController : public EditorViewController {
~CreditCardValidationDelegate() override;
// ValidationDelegate:
- bool ValidateTextfield(views::Textfield* textfield) override;
- bool ValidateCombobox(views::Combobox* combobox) override;
+ bool ValidateTextfield(views::Textfield* textfield,
+ bool display_error) override;
+ bool ValidateCombobox(views::Combobox* combobox,
+ bool display_error) override;
void ComboboxModelChanged(views::Combobox* combobox) override {}
private:
// Validates a specific |value|.
- bool ValidateValue(const base::string16& value);
+ bool ValidateValue(const base::string16& value, bool display_error);
EditorField field_;
// Outlives this class.

Powered by Google App Engine
This is Rietveld 408576698