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

Unified Diff: chrome/browser/ui/autofill/account_chooser_model.cc

Issue 23537014: rAc: Get rid of dialog type in rAc, there is only one type left now. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase 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/account_chooser_model.cc
diff --git a/chrome/browser/ui/autofill/account_chooser_model.cc b/chrome/browser/ui/autofill/account_chooser_model.cc
index 8980c906a3de3ebdaa7e0a2d2d9bc617c9bf3627..b40795ed9b4d98aba255a205d8b1653f407652f8 100644
--- a/chrome/browser/ui/autofill/account_chooser_model.cc
+++ b/chrome/browser/ui/autofill/account_chooser_model.cc
@@ -27,16 +27,14 @@ AccountChooserModelDelegate::~AccountChooserModelDelegate() {}
AccountChooserModel::AccountChooserModel(
AccountChooserModelDelegate* delegate,
PrefService* prefs,
- const AutofillMetrics& metric_logger,
- DialogType dialog_type)
+ const AutofillMetrics& metric_logger)
: ui::SimpleMenuModel(this),
delegate_(delegate),
checked_item_(
prefs->GetBoolean(::prefs::kAutofillDialogPayWithoutWallet) ?
kAutofillItemId : kActiveWalletItemId),
had_wallet_error_(false),
- metric_logger_(metric_logger),
- dialog_type_(dialog_type) {
+ metric_logger_(metric_logger) {
ReconstructMenuItems();
}
@@ -102,7 +100,7 @@ void AccountChooserModel::ExecuteCommand(int command_id, int event_flags) {
chooser_event =
AutofillMetrics::DIALOG_UI_ACCOUNT_CHOOSER_SWITCHED_WALLET_ACCOUNT;
}
- metric_logger_.LogDialogUiEvent(dialog_type_, chooser_event);
+ metric_logger_.LogDialogUiEvent(chooser_event);
checked_item_ = command_id;
ReconstructMenuItems();
« no previous file with comments | « chrome/browser/ui/autofill/account_chooser_model.h ('k') | chrome/browser/ui/autofill/account_chooser_model_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698