Chromium Code Reviews| Index: chrome/browser/password_manager/account_chooser_dialog_android.cc |
| diff --git a/chrome/browser/password_manager/account_chooser_dialog_android.cc b/chrome/browser/password_manager/account_chooser_dialog_android.cc |
| index ce579135d95356ed7e64d1334085ec6cdb86d5f6..8972d7c19b21dd10ab544a7318aa7a4ff8833be4 100644 |
| --- a/chrome/browser/password_manager/account_chooser_dialog_android.cc |
| +++ b/chrome/browser/password_manager/account_chooser_dialog_android.cc |
| @@ -20,6 +20,7 @@ |
| #include "components/browser_sync/browser/profile_sync_service.h" |
| #include "components/password_manager/core/browser/password_bubble_experiment.h" |
| #include "components/password_manager/core/browser/password_manager_constants.h" |
| +#include "components/password_manager/core/browser/password_manager_metrics_util.h" |
| #include "components/password_manager/core/browser/password_ui_utils.h" |
| #include "components/password_manager/core/common/credential_manager_types.h" |
| #include "jni/AccountChooserDialog_jni.h" |
| @@ -207,7 +208,7 @@ void AccountChooserDialogAndroid::WasHidden() { |
| void AccountChooserDialogAndroid::OnDialogCancel() { |
| ChooseCredential(-1, password_manager::CredentialType::CREDENTIAL_TYPE_EMPTY); |
| -} |
| + } |
|
vasilii
2016/05/17 09:45:15
no space
|
| const std::vector<const autofill::PasswordForm*>& |
| AccountChooserDialogAndroid::local_credentials_forms() const { |
| @@ -225,8 +226,13 @@ void AccountChooserDialogAndroid::ChooseCredential( |
| using namespace password_manager; |
| if (type == CredentialType::CREDENTIAL_TYPE_EMPTY) { |
| passwords_data_.ChooseCredential(nullptr); |
| + password_manager::metrics_util::LogAccountChooserUserAction( |
|
vasilii
2016/05/17 09:45:15
indent properly
|
| + password_manager::metrics_util::ACCOUNT_CHOOSER_DISMISSED); |
| + |
| return; |
| } |
| + password_manager::metrics_util::LogAccountChooserUserAction( |
| + password_manager::metrics_util::ACCOUNT_CHOOSER_CREDENTIAL_CHOSEN); |
| DCHECK_EQ(CredentialType::CREDENTIAL_TYPE_PASSWORD, type); |
| const auto& credentials_forms = |
| (type == CredentialType::CREDENTIAL_TYPE_PASSWORD) |