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

Unified Diff: chrome/browser/password_manager/chrome_password_manager_client.cc

Issue 259153004: Password bubble: Give users the option of unblacklisting a site. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 7 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/password_manager/chrome_password_manager_client.cc
diff --git a/chrome/browser/password_manager/chrome_password_manager_client.cc b/chrome/browser/password_manager/chrome_password_manager_client.cc
index 0dbeefbb7b6614becc04f55ca24f7a410d6194f3..3aaa123f29dfd84e48ee6cb3ef5bc675c0a9d1fc 100644
--- a/chrome/browser/password_manager/chrome_password_manager_client.cc
+++ b/chrome/browser/password_manager/chrome_password_manager_client.cc
@@ -111,11 +111,12 @@ void ChromePasswordManagerClient::PasswordWasAutofilled(
manage_passwords_bubble_ui_controller->OnPasswordAutofilled(best_matches);
}
-void ChromePasswordManagerClient::PasswordAutofillWasBlocked() const {
- ManagePasswordsBubbleUIController* manage_passwords_bubble_ui_controller =
+void ChromePasswordManagerClient::PasswordAutofillWasBlocked(
+ const autofill::PasswordFormMap& best_matches) const {
+ ManagePasswordsBubbleUIController* controller =
ManagePasswordsBubbleUIController::FromWebContents(web_contents());
- if (manage_passwords_bubble_ui_controller && IsTheHotNewBubbleUIEnabled())
- manage_passwords_bubble_ui_controller->OnBlacklistBlockedAutofill();
+ if (controller && IsTheHotNewBubbleUIEnabled())
+ controller->OnBlacklistBlockedAutofill(best_matches);
}
void ChromePasswordManagerClient::AuthenticateAutofillAndFillForm(

Powered by Google App Engine
This is Rietveld 408576698