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

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

Issue 2833193002: Trigger Password Protection ping on username/password field on focus (Closed)
Patch Set: nit Created 3 years, 8 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 1a14e357965f7b1db6c5fd4f3138ba20b87b009e..6d4abb6aec386d13b25316347cc908189950d5ac 100644
--- a/chrome/browser/password_manager/chrome_password_manager_client.cc
+++ b/chrome/browser/password_manager/chrome_password_manager_client.cc
@@ -65,7 +65,7 @@
#include "net/base/url_util.h"
#include "third_party/re2/src/re2/re2.h"
-#if defined(SAFE_BROWSING_DB_LOCAL) || defined(SAFE_BROWSING_DB_REMOTE)
+#if defined(SAFE_BROWSING_DB_LOCAL)
#include "chrome/browser/browser_process.h"
#include "chrome/browser/safe_browsing/safe_browsing_service.h"
#include "components/safe_browsing/password_protection/password_protection_service.h"
@@ -410,6 +410,17 @@ ChromePasswordManagerClient::GetPasswordProtectionService() const {
}
return nullptr;
}
+
+void ChromePasswordManagerClient::CheckSafeBrowsingReputation(
+ const GURL& form_action,
+ const GURL& frame_url) {
+ safe_browsing::PasswordProtectionService* pps =
+ GetPasswordProtectionService();
+ if (pps) {
+ pps->MaybeStartLowReputationRequest(GetMainFrameURL(), form_action,
+ frame_url);
+ }
+}
#endif
// TODO(crbug.com/706392): Fix password reuse detection for Android.

Powered by Google App Engine
This is Rietveld 408576698