Update some stale TODO()s in c/b/ui/autofill.

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 1586 matching lines @@
       CvcIconForCreditCardType(credit_card_type);
 
   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.

[Evan Stade, 2013/10/01 01:53:00]

do we have sufficient tests?

[Dan Beam, 2013/10/01 02:00:41]

we have quite a few for InputsAreValid(), but I'll leave for this method.

 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()) {
       TypeErrorInputMap::const_iterator iter = it->second.find(type);
       if (iter != it->second.end()) {
@@ skipping 85 matching lines @@
     default:
       NOTREACHED();  // Trying to validate unknown field.
       break;
   }
 
   return value.empty() ?
       l10n_util::GetStringUTF16(IDS_AUTOFILL_DIALOG_VALIDATION_MISSING_VALUE) :
       base::string16();
 }
 
-// TODO(estade): Replace all the error messages here with more helpful and
-// translateable ones. TODO(groby): Also add tests.
 ValidityData AutofillDialogControllerImpl::InputsAreValid(
     DialogSection section,
     const DetailOutputMap& inputs,
     ValidationType validation_type) {
   ValidityData invalid_messages;
   std::map<ServerFieldType, string16> field_values;
   for (DetailOutputMap::const_iterator iter = inputs.begin();
        iter != inputs.end(); ++iter) {
     const ServerFieldType type = iter->first->type;
 
@@ skipping 446 matching lines @@
 
 void AutofillDialogControllerImpl::OnDidAcceptLegalDocuments() {
   DCHECK(is_submitting_ && IsPayingWithWallet());
   has_accepted_legal_documents_ = true;
   LoadRiskFingerprintData();
 }
 
 void AutofillDialogControllerImpl::OnDidAuthenticateInstrument(bool success) {
   DCHECK(is_submitting_ && IsPayingWithWallet());
 
-  // TODO(dbeam): use the returned full wallet. b/8332329
+  // TODO(dbeam): use the returned full wallet. http://crbug.com/224992
   if (success) {
     GetFullWallet();
   } else {
     DisableWallet(wallet::WalletClient::UNKNOWN_ERROR);
     SuggestionsUpdated();
   }
 }
 
 void AutofillDialogControllerImpl::OnDidGetFullWallet(
     scoped_ptr<wallet::FullWallet> full_wallet) {
@@ skipping 934 matching lines @@
       source_url_,
       wallet_items_->google_transaction_id(),
       capabilities,
       wallet_items_->HasRequiredAction(wallet::SETUP_WALLET)));
 }
 
 void AutofillDialogControllerImpl::HandleSaveOrUpdateRequiredActions(
     const std::vector<wallet::RequiredAction>& required_actions) {
   DCHECK(!required_actions.empty());
 
-  // TODO(ahutter): Invesitigate if we need to support more generic actions on
+  // TODO(ahutter): Investigate if we need to support more generic actions on
   // this call such as GAIA_AUTH. See crbug.com/243457.
   for (std::vector<wallet::RequiredAction>::const_iterator iter =
            required_actions.begin();
        iter != required_actions.end(); ++iter) {
     if (*iter != wallet::INVALID_FORM_FIELD) {
       // TODO(dbeam): handle this more gracefully.
       DisableWallet(wallet::WalletClient::UNKNOWN_ERROR);
     }
   }
   SetIsSubmitting(false);
@@ skipping 292 matching lines @@
   net::URLRequestContextGetter* request_context = profile_->GetRequestContext();
   signin_helper_.reset(new wallet::WalletSigninHelper(this, request_context));
   signin_helper_->StartWalletCookieValueFetch();
 
   username_fetcher_.reset(
       new wallet::WalletSigninHelper(this, request_context));
   username_fetcher_->StartUserNameFetch();
 }
 
 }  // namespace autofill