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

Unified Diff: components/password_manager/content/browser/content_password_manager_driver.cc

Issue 2055633003: [Password Generation] Sends form classifier vote to autofill server (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@pg_form_classification
Patch Set: Changed enum values Created 4 years, 6 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: components/password_manager/content/browser/content_password_manager_driver.cc
diff --git a/components/password_manager/content/browser/content_password_manager_driver.cc b/components/password_manager/content/browser/content_password_manager_driver.cc
index c53e176a546787f10adde0c659cd0e12b90604e2..88419ead623fb9306ad650de889dff7f94a83dc7 100644
--- a/components/password_manager/content/browser/content_password_manager_driver.cc
+++ b/components/password_manager/content/browser/content_password_manager_driver.cc
@@ -162,6 +162,8 @@ bool ContentPasswordManagerDriver::HandleMessage(const IPC::Message& message) {
IPC_MESSAGE_HANDLER(AutofillHostMsg_InPageNavigation, OnInPageNavigation)
IPC_MESSAGE_HANDLER(AutofillHostMsg_PresaveGeneratedPassword,
OnPresaveGeneratedPassword)
+ IPC_MESSAGE_HANDLER(AutofillHostMsg_SaveFormClassifierVote,
+ OnSaveFormClassifierVote)
IPC_MESSAGE_HANDLER(AutofillHostMsg_PasswordNoLongerGenerated,
OnPasswordNoLongerGenerated)
IPC_MESSAGE_HANDLER(AutofillHostMsg_FocusedPasswordFormFound,
@@ -261,6 +263,16 @@ void ContentPasswordManagerDriver::OnPasswordNoLongerGenerated(
false);
}
+void ContentPasswordManagerDriver::OnSaveFormClassifierVote(
+ const autofill::PasswordForm& password_form,
+ const base::string16& generation_field) {
+ if (!CheckChildProcessSecurityPolicy(
+ password_form.origin,
+ BadMessageReason::CPMD_BAD_ORIGIN_SAVE_FORM_CLASSIFIER_VOTE))
+ return;
+ GetPasswordManager()->SaveFormClassifierVote(password_form, generation_field);
+}
+
bool ContentPasswordManagerDriver::CheckChildProcessSecurityPolicy(
const GURL& url,
BadMessageReason reason) {

Powered by Google App Engine
This is Rietveld 408576698