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

Side by Side Diff: chrome/browser/ui/views/payments/credit_card_editor_view_controller.h

Issue 2803443003: [Payments] Added region load failure tolerance and tests to PR editor. (Closed)
Patch Set: Rename function post-rebase Created 3 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_VIEWS_PAYMENTS_CREDIT_CARD_EDITOR_VIEW_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_PAYMENTS_CREDIT_CARD_EDITOR_VIEW_CONTROLLER_H_
6 #define CHROME_BROWSER_UI_VIEWS_PAYMENTS_CREDIT_CARD_EDITOR_VIEW_CONTROLLER_H_ 6 #define CHROME_BROWSER_UI_VIEWS_PAYMENTS_CREDIT_CARD_EDITOR_VIEW_CONTROLLER_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 // passed in to validate |field| (the data will be copied to the delegate). 61 // passed in to validate |field| (the data will be copied to the delegate).
62 CreditCardValidationDelegate( 62 CreditCardValidationDelegate(
63 const EditorField& field, 63 const EditorField& field,
64 EditorViewController* controller, 64 EditorViewController* controller,
65 const std::vector<std::string>& supported_card_networks); 65 const std::vector<std::string>& supported_card_networks);
66 ~CreditCardValidationDelegate() override; 66 ~CreditCardValidationDelegate() override;
67 67
68 // ValidationDelegate: 68 // ValidationDelegate:
69 bool ValidateTextfield(views::Textfield* textfield) override; 69 bool ValidateTextfield(views::Textfield* textfield) override;
70 bool ValidateCombobox(views::Combobox* combobox) override; 70 bool ValidateCombobox(views::Combobox* combobox) override;
71 void ComboboxModelChanged(views::Combobox* combobox) override{};
please use gerrit instead 2017/04/11 21:05:35 Please add a space before "{}" and remove the ";".
MAD 2017/04/12 02:46:07 I was wondering why I kept changing this, and I ju
please use gerrit instead 2017/04/12 14:01:17 git cl format does this? Yikes! Please file a bug
MAD 2017/04/12 15:50:12 D'Ho! I realized that I forgot to remove the semi-
71 72
72 private: 73 private:
73 // Validates a specific |value|. 74 // Validates a specific |value|.
74 bool ValidateValue(const base::string16& value); 75 bool ValidateValue(const base::string16& value);
75 76
76 EditorField field_; 77 EditorField field_;
77 // Outlives this class. 78 // Outlives this class.
78 EditorViewController* controller_; 79 EditorViewController* controller_;
79 // The list of supported basic card networks. 80 // The list of supported basic card networks.
80 std::set<std::string> supported_card_networks_; 81 std::set<std::string> supported_card_networks_;
81 82
82 DISALLOW_COPY_AND_ASSIGN(CreditCardValidationDelegate); 83 DISALLOW_COPY_AND_ASSIGN(CreditCardValidationDelegate);
83 }; 84 };
84 85
85 // If non-nullptr, a pointer to an object to be edited. Must outlive this 86 // If non-nullptr, a pointer to an object to be edited. Must outlive this
86 // controller. 87 // controller.
87 autofill::CreditCard* credit_card_to_edit_; 88 autofill::CreditCard* credit_card_to_edit_;
88 89
89 DISALLOW_COPY_AND_ASSIGN(CreditCardEditorViewController); 90 DISALLOW_COPY_AND_ASSIGN(CreditCardEditorViewController);
90 }; 91 };
91 92
92 } // namespace payments 93 } // namespace payments
93 94
94 #endif // CHROME_BROWSER_UI_VIEWS_PAYMENTS_CREDIT_CARD_EDITOR_VIEW_CONTROLLER_H _ 95 #endif // CHROME_BROWSER_UI_VIEWS_PAYMENTS_CREDIT_CARD_EDITOR_VIEW_CONTROLLER_H _
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698