Add corresponding bugs to TODO()s and file a few more to track and prioritize

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 <string>
 
 #include "base/bind.h"
@@ skipping 779 matching lines @@
 
   if (IsPayingWithWallet()) {
     int index;
     bool success = base::StringToInt(item_key, &index);
     DCHECK(success);
 
     if (section == SECTION_CC_BILLING) {
       return scoped_ptr<DataModelWrapper>(
           new WalletInstrumentWrapper(wallet_items_->instruments()[index]));
     }
-    // TODO(dbeam): should SECTION_EMAIL get here?
+    // TODO(dbeam): should SECTION_EMAIL get here? http://crbug.com/223923
     return scoped_ptr<DataModelWrapper>(
         new WalletAddressWrapper(wallet_items_->addresses()[index]));
   }
 
   if (section == SECTION_CC) {
     CreditCard* card = GetManager()->GetCreditCardByGUID(item_key);
     DCHECK(card);
     return scoped_ptr<DataModelWrapper>(new AutofillCreditCardWrapper(card));
   }
 
@@ skipping 613 matching lines @@
   NOTIMPLEMENTED();
 }
 
 void AutofillDialogControllerImpl::OnDidGetWalletItems(
     scoped_ptr<wallet::WalletItems> wallet_items) {
   DCHECK(account_chooser_model_.WalletIsSelected());
 
   legal_documents_text_.clear();
   legal_document_link_ranges_.clear();
 
-  // TODO(dbeam): verify all items support kCartCurrency?
+  // TODO(dbeam): verify items support kCartCurrency? http://crbug.com/232952
   wallet_items_ = wallet_items.Pass();
   OnWalletOrSigninUpdate();
 }
 
 void AutofillDialogControllerImpl::OnDidSaveAddress(
     const std::string& address_id,
     const std::vector<wallet::RequiredAction>& required_actions) {
-  // TODO(dbeam): handle required actions.
+  // TODO(dbeam): handle required actions. http://crbug.com/163508
   active_address_id_ = address_id;
   GetFullWallet();
 }
 
 void AutofillDialogControllerImpl::OnDidSaveInstrument(
     const std::string& instrument_id,
     const std::vector<wallet::RequiredAction>& required_actions) {
   DCHECK(is_submitting_ && IsPayingWithWallet());
 
-  // TODO(dbeam): handle required actions.
+  // TODO(dbeam): handle required actions. http://crbug.com/163508
   active_instrument_id_ = instrument_id;
   GetFullWallet();
 }
 
 void AutofillDialogControllerImpl::OnDidSaveInstrumentAndAddress(
     const std::string& instrument_id,
     const std::string& address_id,
     const std::vector<wallet::RequiredAction>& required_actions) {
   DCHECK(is_submitting_ && IsPayingWithWallet());
 
-  // TODO(dbeam): handle required actions.
+  // TODO(dbeam): handle required actions. http://crbug.com/163508
   active_instrument_id_ = instrument_id;
   active_address_id_ = address_id;
   GetFullWallet();
 }
 
 void AutofillDialogControllerImpl::OnDidUpdateAddress(
     const std::string& address_id,
     const std::vector<wallet::RequiredAction>& required_actions) {
   DCHECK(is_submitting_ && IsPayingWithWallet());
 
-  // TODO(dbeam): Handle this callback.
+  // TODO(dbeam): Handle this callback. http://crbug.com/163508
   NOTIMPLEMENTED() << " address_id=" << address_id;
 }
 
 void AutofillDialogControllerImpl::OnDidUpdateInstrument(
     const std::string& instrument_id,
     const std::vector<wallet::RequiredAction>& required_actions) {
   DCHECK(is_submitting_ && IsPayingWithWallet());
 
-  // TODO(dbeam): handle required actions.
+  // TODO(dbeam): handle required actions. http://crbug.com/163508
 }
 
 void AutofillDialogControllerImpl::OnWalletError(
     wallet::WalletClient::ErrorType error_type) {
-  // TODO(dbeam): Do something with |error_type|.
+  // TODO(dbeam): Do something with |error_type|. http://crbug.com/164410
   DisableWallet();
 }
 
 void AutofillDialogControllerImpl::OnMalformedResponse() {
   DisableWallet();
 }
 
 void AutofillDialogControllerImpl::OnNetworkError(int response_code) {
   DisableWallet();
 }
@@ skipping 131 matching lines @@
   suggested_cc_billing_.Reset();
   suggested_shipping_.Reset();
   HidePopup();
 
   if (IsPayingWithWallet()) {
     // TODO(estade): fill in the email address.
 
     const std::vector<wallet::Address*>& addresses =
         wallet_items_->addresses();
     for (size_t i = 0; i < addresses.size(); ++i) {
-      // TODO(dbeam): respect wallet_items_->default_instrument_id().
+      // TODO(dbeam): respect the default instrument ID. http://crbug.com/232954
       suggested_shipping_.AddKeyedItemWithSublabel(
           base::IntToString(i),
           addresses[i]->DisplayName(),
           addresses[i]->DisplayNameDetail());
     }
 
     if (!IsSubmitPausedOn(wallet::VERIFY_CVV)) {
       const std::vector<wallet::WalletItems::MaskedInstrument*>& instruments =
           wallet_items_->instruments();
       for (size_t i = 0; i < instruments.size(); ++i) {
-        // TODO(dbeam): respect wallet_items_->default_address_id().
+        // TODO(dbeam): respect the default address ID. http://crbug.com/232954
         suggested_cc_billing_.AddKeyedItemWithSublabelAndIcon(
             base::IntToString(i),
             instruments[i]->DisplayName(),
             instruments[i]->DisplayNameDetail(),
             instruments[i]->CardIcon());
       }
 
       suggested_cc_billing_.AddKeyedItem(
           std::string(),
           l10n_util::GetStringUTF16(IDS_AUTOFILL_DIALOG_ADD_BILLING_DETAILS));
@@ skipping 196 matching lines @@
 }
 
 void AutofillDialogControllerImpl::HidePopup() {
   if (popup_controller_)
     popup_controller_->Hide();
   input_showing_popup_ = NULL;
 }
 
 void AutofillDialogControllerImpl::LoadRiskFingerprintData() {
   // TODO(dbeam): Add a CHECK or otherwise strong guarantee that the ToS have
-  // been accepted prior to calling into this method.  Also, ensure that the UI
+  // been accepted prior to calling into this method. Also, ensure that the UI
   // contains a clear indication to the user as to what data will be collected.
-  // Until then, this code should not be called.
+  // Until then, this code should not be called. http://crbug.com/173505
 
   int64 gaia_id = 0;
   bool success =
       base::StringToInt64(wallet_items_->obfuscated_gaia_id(), &gaia_id);
   DCHECK(success);
 
   gfx::Rect window_bounds =
       GetBaseWindowForWebContents(web_contents())->GetBounds();
 
   PrefService* user_prefs = profile_->GetPrefs();
@@ skipping 77 matching lines @@
   is_submitting_ = submitting;
 
   if (view_) {
     view_->UpdateButtonStrip();
     view_->UpdateNotificationArea();
   }
 }
 
 void AutofillDialogControllerImpl::SubmitWithWallet() {
   // TODO(dbeam): disallow interacting with the dialog while submitting.
+  // http://crbug.com/230932
 
   active_instrument_id_.clear();
   active_address_id_.clear();
   full_wallet_.reset();
 
   if (!section_editing_state_[SECTION_CC_BILLING]) {
     SuggestionsMenuModel* billing =
         SuggestionsMenuModelForSection(SECTION_CC_BILLING);
     if (!billing->GetItemKeyForCheckedItem().empty() &&
         billing->checked_item() <
             static_cast<int>(wallet_items_->instruments().size())) {
       const wallet::WalletItems::MaskedInstrument* active_instrument =
           wallet_items_->instruments()[billing->checked_item()];
       active_instrument_id_ = active_instrument->object_id();
 
-      // TODO(dbeam): does re-using instrument address IDs work?
+      // TODO(dbeam): save this as a shipping address. http://crbug.com/225442
       if (ShouldUseBillingForShipping())
         active_address_id_ = active_instrument->address().object_id();
     }
   }
 
   if (!section_editing_state_[SECTION_SHIPPING] && active_address_id_.empty()) {
     SuggestionsMenuModel* shipping =
         SuggestionsMenuModelForSection(SECTION_SHIPPING);
     if (!shipping->GetItemKeyForCheckedItem().empty() &&
         shipping->checked_item() <
@@ skipping 192 matching lines @@
         AutofillMetrics::DIALOG_USER_SIGNED_IN_NO_WALLET_NO_AUTOFILL;
   }
 
   // Has Wallet items.
   return has_autofill_profiles ?
       AutofillMetrics::DIALOG_USER_SIGNED_IN_HAS_WALLET_HAS_AUTOFILL :
       AutofillMetrics::DIALOG_USER_SIGNED_IN_HAS_WALLET_NO_AUTOFILL;
 }
 
 }  // namespace autofill