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

Unified Diff: chrome/browser/autofill/autofill_dialog_controller_mac.mm

Issue 3806008: Never allow empty preview entries. As some data is always present (we drop co... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 2 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
« no previous file with comments | « no previous file | chrome/browser/autofill/autofill_dialog_gtk.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autofill/autofill_dialog_controller_mac.mm
===================================================================
--- chrome/browser/autofill/autofill_dialog_controller_mac.mm (revision 63786)
+++ chrome/browser/autofill/autofill_dialog_controller_mac.mm (working copy)
@@ -32,20 +32,6 @@
// dialog.
typedef std::map<Profile*, AutoFillDialogController*> ProfileControllerMap;
-// Update profile labels passed as |input|. When profile data changes as a
-// result of adding new profiles, edititing existing profiles, or deleting a
-// profile, then the list of profiles need to have their derived labels
-// recomputed.
-void UpdateProfileLabels(std::vector<AutoFillProfile>* input) {
- DCHECK(input);
- std::vector<AutoFillProfile*> profiles;
- profiles.resize(input->size());
- for (size_t i = 0; i < input->size(); ++i) {
- profiles[i] = &(*input)[i];
- }
- AutoFillProfile::AdjustInferredLabels(&profiles);
-}
-
} // namespace
// Delegate protocol that needs to be in place for the AutoFillTableView's
@@ -839,7 +825,6 @@
iter != creditCards.end(); ++iter)
creditCards_.push_back(**iter);
- UpdateProfileLabels(&profiles_);
[tableView_ reloadData];
}
« no previous file with comments | « no previous file | chrome/browser/autofill/autofill_dialog_gtk.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698