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

Unified Diff: chrome/browser/ui/autofill/autofill_dialog_view_delegate.h

Issue 23756007: [rac] Show amex specific cvc hint (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move icon dependency logic into controller for testability Created 7 years, 3 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/autofill/autofill_dialog_view_delegate.h
diff --git a/chrome/browser/ui/autofill/autofill_dialog_view_delegate.h b/chrome/browser/ui/autofill/autofill_dialog_view_delegate.h
index 8d5ad98fe1df3463ccb46740c2522fa2217fb071..5e1143f4bf9d1f89bf18859d8e7c96dddbc249c0 100644
--- a/chrome/browser/ui/autofill/autofill_dialog_view_delegate.h
+++ b/chrome/browser/ui/autofill/autofill_dialog_view_delegate.h
@@ -124,6 +124,18 @@ class AutofillDialogViewDelegate {
virtual gfx::Image IconForField(ServerFieldType type,
const string16& user_input) const = 0;
+ // Returns true if the icon of this field type |depdendent_type| depends on
+ // the value of another field. For example, returns true for CVC, because CVC
+ // hint images depend on credit card type.
+ virtual bool IsIconDependentField(ServerFieldType dependent_type) const = 0;
+
+ // Returns true if the value of this field type |dependency_type| controls the
+ // icon of another field. Sets |dependent_type| to the type of the dependent
+ // field. For example, returns true for credit card number, because some
+ // credit card types have different CVC hint images.
+ virtual bool IsIconDependencyField(ServerFieldType dependency_type,
+ ServerFieldType* dependent_type) const = 0;
+
// Decides whether input of |value| is valid for a field of type |type|. If
// valid, the returned string will be empty. Otherwise it will contain an
// error message.

Powered by Google App Engine
This is Rietveld 408576698