Index: chrome/browser/ui/passwords/manage_passwords_bubble_model.cc |
diff --git a/chrome/browser/ui/passwords/manage_passwords_bubble_model.cc b/chrome/browser/ui/passwords/manage_passwords_bubble_model.cc |
index cb41abdcf9b28016a50afc75f697097ff9a0ba8f..fe5d7697264e598bacd16a140c45a61db72a027f 100644 |
--- a/chrome/browser/ui/passwords/manage_passwords_bubble_model.cc |
+++ b/chrome/browser/ui/passwords/manage_passwords_bubble_model.cc |
@@ -182,6 +182,8 @@ void ManagePasswordsBubbleModel::OnBubbleShown( |
metrics_util::AUTOMATIC_GENERATED_PASSWORD_CONFIRMATION; |
} else if (state_ == password_manager::ui::CREDENTIAL_REQUEST_STATE) { |
display_disposition_ = metrics_util::AUTOMATIC_CREDENTIAL_REQUEST; |
+ } else if (state_ == password_manager::ui::AUTO_SIGNIN_STATE) { |
+ display_disposition_ = metrics_util::AUTOMATIC_SIGNIN_TOAST; |
} else { |
display_disposition_ = metrics_util::AUTOMATIC_WITH_PASSWORD_PENDING; |
} |
@@ -230,13 +232,13 @@ void ManagePasswordsBubbleModel::OnBubbleHidden() { |
metrics_util::LogUIDismissalReason(dismissal_reason_); |
// Other use cases have been reported in the callbacks like OnSaveClicked(). |
- if (dismissal_reason_ == metrics_util::NO_DIRECT_INTERACTION) |
+ if (state_ == password_manager::ui::PENDING_PASSWORD_STATE && |
+ dismissal_reason_ == metrics_util::NO_DIRECT_INTERACTION) |
RecordExperimentStatistics(web_contents(), dismissal_reason_); |
} |
void ManagePasswordsBubbleModel::OnCollectURLClicked(const std::string& url) { |
dismissal_reason_ = metrics_util::CLICKED_COLLECT_URL; |
- RecordExperimentStatistics(web_contents(), dismissal_reason_); |
// User interaction with bubble has happened, do not need to show bubble |
// in case it was before transition to another page. |
state_ = password_manager::ui::ASK_USER_REPORT_URL_BUBBLE_SHOWN_STATE; |
@@ -249,7 +251,6 @@ void ManagePasswordsBubbleModel::OnCollectURLClicked(const std::string& url) { |
void ManagePasswordsBubbleModel::OnDoNotCollectURLClicked() { |
dismissal_reason_ = metrics_util::CLICKED_DO_NOT_COLLECT_URL; |
- RecordExperimentStatistics(web_contents(), dismissal_reason_); |
// User interaction with bubble has happened, do not need to show bubble |
// in case it was before transition to another page. |
state_ = password_manager::ui::ASK_USER_REPORT_URL_BUBBLE_SHOWN_STATE; |
@@ -314,6 +315,10 @@ void ManagePasswordsBubbleModel::OnManageLinkClicked() { |
->NavigateToPasswordManagerSettingsPage(); |
} |
+void ManagePasswordsBubbleModel::OnAutoSignInToastTimeout() { |
+ dismissal_reason_ = metrics_util::AUTO_SIGNIN_TOAST_TIMEOUT; |
+} |
+ |
void ManagePasswordsBubbleModel::OnPasswordAction( |
const autofill::PasswordForm& password_form, |
PasswordAction action) { |