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

Unified Diff: chrome/browser/ui/passwords/manage_passwords_bubble_model.cc

Issue 765073004: Adding skeleton code for showing Bubble, asking user to submit URL when Password Manager fails (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years 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/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 d572f6e3cc078e4ebffe186b539f8d326e7d59ee..bf4822b5a7bbecf0bded216f780083d597f21e9a 100644
--- a/chrome/browser/ui/passwords/manage_passwords_bubble_model.cc
+++ b/chrome/browser/ui/passwords/manage_passwords_bubble_model.cc
@@ -54,6 +54,7 @@ ManagePasswordsBubbleModel::ManagePasswordsBubbleModel(
ManagePasswordsUIController* controller =
ManagePasswordsUIController::FromWebContents(web_contents);
+ origin_ = controller->origin();
state_ = controller->state();
if (password_manager::ui::IsPendingState(state_))
pending_password_ = controller->PendingPassword();
@@ -71,6 +72,9 @@ ManagePasswordsBubbleModel::ManagePasswordsBubbleModel(
l10n_util::GetStringUTF16(IDS_MANAGE_PASSWORDS_CONFIRM_GENERATED_TITLE);
} else if (password_manager::ui::IsCredentialsState(state_)) {
title_ = l10n_util::GetStringUTF16(IDS_MANAGE_PASSWORDS_CHOOSE_TITLE);
+ } else if (state_ == password_manager::ui::ASK_USER_TO_REPORT_URL_STATE) {
+ title_ =
+ l10n_util::GetStringUTF16(IDS_MANAGE_PASSWORDS_ASK_TO_SUBMIT_URL_TITLE);
} else {
title_ = l10n_util::GetStringUTF16(IDS_MANAGE_PASSWORDS_TITLE);
}
@@ -141,6 +145,17 @@ void ManagePasswordsBubbleModel::OnBubbleHidden() {
RecordExperimentStatistics(web_contents(), dismissal_reason_);
}
+void ManagePasswordsBubbleModel::OnCollectURLClicked() {
+ dismissal_reason_ = metrics_util::CLICKED_COLLECT_URL;
+ RecordExperimentStatistics(web_contents(), dismissal_reason_);
+ // TODO(melandory): Implement feedback sending logic.
+}
+
+void ManagePasswordsBubbleModel::OnDoNotCollectURLClicked() {
+ dismissal_reason_ = metrics_util::CLICKED_DO_NOT_COLLECT_URL;
+ RecordExperimentStatistics(web_contents(), dismissal_reason_);
+}
+
void ManagePasswordsBubbleModel::OnNopeClicked() {
dismissal_reason_ = metrics_util::CLICKED_NOPE;
RecordExperimentStatistics(web_contents(), dismissal_reason_);

Powered by Google App Engine
This is Rietveld 408576698