[rac] Show amex specific cvc hint

chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc

 // Copyright 2013 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/autofill/autofill_dialog_controller_impl.h"
 
 #include <algorithm>
 #include <map>
 #include <string>
 
@@ skipping 1489 matching lines @@
 gfx::Image AutofillDialogControllerImpl::SuggestionIconForSection(
     DialogSection section) {
   scoped_ptr<DataModelWrapper> model = CreateWrapper(section);
   if (!model.get())
     return gfx::Image();
 
   return model->GetIcon();
 }
 
 gfx::Image AutofillDialogControllerImpl::ExtraSuggestionIconForSection(
-    DialogSection section) const {
-  if (section == SECTION_CC || section == SECTION_CC_BILLING)
-    return IconForField(CREDIT_CARD_VERIFICATION_CODE, string16());
-
-  return gfx::Image();
+    DialogSection section) {
+  if (section != SECTION_CC && section != SECTION_CC_BILLING)
+    return gfx::Image();
+  scoped_ptr<DataModelWrapper> model = CreateWrapper(section);
+  if (!model.get())
+    return gfx::Image();
+  FieldValueMap section_values;
+  section_values[CREDIT_CARD_NUMBER] =
+      model->GetInfo(AutofillType(CREDIT_CARD_NUMBER));

[Evan Stade, 2013/09/16 23:36:59]

this won't work for wallet (SECTION_CC_BILLING); we don't know the user's actual
CC number. The instrument does know the type of card, however.

[please use gerrit instead, 2013/09/17 00:38:17]

Using CREDIT_CARD_TYPE instead.

+  section_values[CREDIT_CARD_VERIFICATION_CODE] = string16();
+  FieldIconMap section_icons = IconsForFields(section_values);
+  return section_icons[CREDIT_CARD_VERIFICATION_CODE];

[Evan Stade, 2013/09/16 23:36:59]

this function needs vertical whitespace

[please use gerrit instead, 2013/09/17 00:38:17]

Done.

 }
 
+// TODO(groby): Remove this deprecated method after Mac starts using
+// IconsForFields. http://crbug.com/292876
 gfx::Image AutofillDialogControllerImpl::IconForField(
     ServerFieldType type, const string16& user_input) const {
   ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
   if (type == CREDIT_CARD_VERIFICATION_CODE)
     return rb.GetImageNamed(IDR_CREDIT_CARD_CVC_HINT);
 
   if (type == CREDIT_CARD_NUMBER) {
     const int input_card_idr = CreditCard::IconResourceId(
         CreditCard::GetCreditCardType(user_input));
     if (input_card_idr != IDR_AUTOFILL_CC_GENERIC)
       return rb.GetImageNamed(input_card_idr);
 
     // When the credit card type is unknown, no image should be shown. However,
     // to simplify the view code on Mac, save space for the credit card image by
     // returning a transparent image of the appropriate size.
     gfx::ImageSkia image = *rb.GetImageSkiaNamed(input_card_idr);
     return
         gfx::Image(gfx::ImageSkiaOperations::CreateTransparentImage(image, 0));
   }
 
   return gfx::Image();
 }
 
+FieldIconMap AutofillDialogControllerImpl::IconsForFields(
+    const FieldValueMap& user_inputs) const {
+  FieldIconMap result;
+  ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
+  FieldValueMap::const_iterator credit_card_iter =
+      user_inputs.find(CREDIT_CARD_NUMBER);
+  std::string credit_card_type = autofill::kGenericCard;
+  if (credit_card_iter != user_inputs.end()) {
+    const string16& credit_card_number = credit_card_iter->second;
+    credit_card_type = CreditCard::GetCreditCardType(credit_card_number);
+    const int input_card_idr = CreditCard::IconResourceId(credit_card_type);
+    result[CREDIT_CARD_NUMBER] = rb.GetImageNamed(input_card_idr);
+    if (input_card_idr == IDR_AUTOFILL_CC_GENERIC) {
+      // When the credit card type is unknown, no image should be shown.
+      // However, to simplify the view code on Mac, save space for the credit
+      // card image by returning a transparent image of the appropriate size.
+      result[CREDIT_CARD_NUMBER] =
+          gfx::Image(gfx::ImageSkiaOperations::CreateTransparentImage(
+              result[CREDIT_CARD_NUMBER].AsImageSkia(), 0));
+    }
+  }
+
+  FieldValueMap::const_iterator cvc_iter =
+      user_inputs.find(CREDIT_CARD_VERIFICATION_CODE);
+  if (cvc_iter == user_inputs.end())
+    return result;
+  if (credit_card_type == autofill::kAmericanExpressCard) {
+    result[CREDIT_CARD_VERIFICATION_CODE] =
+        rb.GetImageNamed(IDR_CREDIT_CARD_CVC_HINT_AMEX);
+    return result;
+  }
+  result[CREDIT_CARD_VERIFICATION_CODE] =
+      rb.GetImageNamed(IDR_CREDIT_CARD_CVC_HINT);
+  return result;
+}
+
+bool AutofillDialogControllerImpl::FieldControlsIcons(
+    ServerFieldType type) const {
+  return type == CREDIT_CARD_NUMBER;
+}
+
 // TODO(estade): Replace all the error messages here with more helpful and
 // translateable ones. TODO(groby): Also add tests.
 string16 AutofillDialogControllerImpl::InputValidityMessage(
     DialogSection section,
     ServerFieldType type,
     const string16& value) {
   // If the field is edited, clear any Wallet errors.
   if (IsPayingWithWallet()) {
     WalletValidationErrors::iterator it = wallet_errors_.find(section);
     if (it != wallet_errors_.end()) {
@@ skipping 1810 matching lines @@
 }
 
 void AutofillDialogControllerImpl::OnSubmitButtonDelayEnd() {
   if (!view_)
     return;
   ScopedViewUpdates updates(view_.get());
   view_->UpdateButtonStrip();
 }
 
 }  // namespace autofill