[Payments] Implements the credit card editor for Desktop UI

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 <memory>
+#include <vector>

[please use gerrit instead, 2017/02/02 20:19:02]

Generally it's not necessary to add includes for the types that are used only to
override method declartions in the parent class.

[Mathieu, 2017/02/03 02:18:57]

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.

[please use gerrit instead, 2017/02/02 20:19:02]

Love this comment! :-D

[Mathieu, 2017/02/03 02:18:57]

got it from a wise person

+  CreditCardEditorViewController(PaymentRequest* request,
+                                 PaymentRequestDialogView* dialog);
+  ~CreditCardEditorViewController() override;
+
+  // EditorViewController implementation.
+  std::vector<std::unique_ptr<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_