[Payments] Basic validation in the credit card editor.

chrome/browser/ui/views/payments/credit_card_editor_view_controller.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_PAYMENTS_CREDIT_CARD_EDITOR_VIEW_CONTROLLER_H_
 #define CHROME_BROWSER_UI_VIEWS_PAYMENTS_CREDIT_CARD_EDITOR_VIEW_CONTROLLER_H_
 
+#include <vector>

[please use gerrit instead, 2017/02/07 21:55:22]

Generally it's not necessary to include heads when overriding methods from a
superclass. It's the responsibility of the superclass header file to include
those.

[Mathieu, 2017/02/08 02:03:25]

Done.

+
 #include "base/macros.h"
 #include "chrome/browser/ui/views/payments/editor_view_controller.h"
 
 namespace payments {
 
 class PaymentRequest;
 class PaymentRequestDialogView;
 
 // Credit card editor screen of the Payment Request flow.
 class CreditCardEditorViewController : public EditorViewController {
  public:
   // Does not take ownership of the arguments, which should outlive this object.
   CreditCardEditorViewController(PaymentRequest* request,
                                  PaymentRequestDialogView* dialog);
   ~CreditCardEditorViewController() override;
 
   // EditorViewController implementation.
   std::vector<EditorField> GetFieldDefinitions() override;
   bool ValidateModelAndSave() override;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(CreditCardEditorViewController);
 };
 
 }  // namespace payments
 
 #endif  // CHROME_BROWSER_UI_VIEWS_PAYMENTS_CREDIT_CARD_EDITOR_VIEW_CONTROLLER_H_