Focus first invalid field of payment request editor

chrome/browser/ui/views/payments/contact_info_editor_view_controller.cc

 // 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.
 
 #include "chrome/browser/ui/views/payments/contact_info_editor_view_controller.h"
 
+#include <utility>
+
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/ui/views/payments/validating_textfield.h"
 #include "components/autofill/core/browser/autofill_country.h"
 #include "components/autofill/core/browser/autofill_profile.h"
 #include "components/autofill/core/browser/autofill_type.h"
 #include "components/autofill/core/browser/personal_data_manager.h"
 #include "components/autofill/core/browser/validation.h"
 #include "components/autofill/core/common/autofill_constants.h"
 #include "components/payments/content/payment_request_spec.h"
 #include "components/payments/content/payment_request_state.h"
@@ skipping 126 matching lines @@
 ContactInfoEditorViewController::ContactInfoValidationDelegate::
     ContactInfoValidationDelegate(const EditorField& field,
                                   const std::string& locale,
                                   EditorViewController* controller)
     : field_(field), controller_(controller), locale_(locale) {}
 
 ContactInfoEditorViewController::ContactInfoValidationDelegate::
     ~ContactInfoValidationDelegate() {}
 
 bool ContactInfoEditorViewController::ContactInfoValidationDelegate::
-    ValidateTextfield(views::Textfield* textfield) {
+    IsValidTextfield(views::Textfield* textfield) {
+  return ValidateTextfield(textfield, nullptr);
+}
+
+bool ContactInfoEditorViewController::ContactInfoValidationDelegate::
+    TextfieldValueChanged(views::Textfield* textfield) {
+  base::string16 error_message;
+  bool is_valid = ValidateTextfield(textfield, &error_message);
+  controller_->DisplayErrorMessageForField(field_, error_message);
+  return is_valid;
+}
+
+bool ContactInfoEditorViewController::ContactInfoValidationDelegate::
+    ValidateTextfield(views::Textfield* textfield,
+                      base::string16* error_message) {
   bool is_valid = true;
-  base::string16 error_message;
 
   if (textfield->text().empty()) {
     is_valid = false;
-    error_message = l10n_util::GetStringUTF16(
-        IDS_PAYMENTS_FIELD_REQUIRED_VALIDATION_MESSAGE);
+    if (error_message) {
+      *error_message = l10n_util::GetStringUTF16(
+          IDS_PAYMENTS_FIELD_REQUIRED_VALIDATION_MESSAGE);
+    }
   } else {
     switch (field_.type) {
       case autofill::PHONE_HOME_WHOLE_NUMBER: {
         const std::string default_region_code =
             autofill::AutofillCountry::CountryCodeForLocale(locale_);
         if (!autofill::IsValidPhoneNumber(textfield->text(),
                                           default_region_code)) {
           is_valid = false;
-          error_message = l10n_util::GetStringUTF16(
-              IDS_PAYMENTS_PHONE_INVALID_VALIDATION_MESSAGE);
+          if (error_message) {
+            *error_message = l10n_util::GetStringUTF16(
+                IDS_PAYMENTS_PHONE_INVALID_VALIDATION_MESSAGE);
+          }
         }
         break;
       }
 
       case autofill::EMAIL_ADDRESS: {
         if (!autofill::IsValidEmailAddress(textfield->text())) {
           is_valid = false;
-          error_message = l10n_util::GetStringUTF16(
-              IDS_PAYMENTS_EMAIL_INVALID_VALIDATION_MESSAGE);
+          if (error_message) {
+            *error_message = l10n_util::GetStringUTF16(
+                IDS_PAYMENTS_EMAIL_INVALID_VALIDATION_MESSAGE);
+          }
         }
         break;
       }
 
       case autofill::NAME_FULL: {
         // We have already determined that name is nonempty, which is the only
         // requirement.
         break;
       }
 
       default: {
         NOTREACHED();
         break;
       }
     }
   }
 
-  controller_->DisplayErrorMessageForField(field_, error_message);
   return is_valid;
 }
 
 bool ContactInfoEditorViewController::ContactInfoValidationDelegate::
-    ValidateCombobox(views::Combobox* combobox) {
+    IsValidCombobox(views::Combobox* combobox) {
   // This UI doesn't contain any comboboxes.
   NOTREACHED();
   return true;
+}
+
+bool ContactInfoEditorViewController::ContactInfoValidationDelegate::
+    ComboboxValueChanged(views::Combobox* combobox) {
+  // This UI doesn't contain any comboboxes.
+  NOTREACHED();
+  return true;
 }
 
 }  // namespace payments