Trigger Password Protection ping on username/password field on focus

chrome/browser/password_manager/chrome_password_manager_client.cc

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..1c262f3a75abc7645ca0a9eb611ccc638236555d 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,16 @@ 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,

[estark, 2017/05/02 14:09:38]

optional nit: I usually see curly braces around bodies that span >1 line

[Jialiu Lin, 2017/05/02 17:36:00]

You're right. curly braces are needed here. Thanks for catching this. 

+                                        frame_url);
+}
 #endif
 
 // TODO(crbug.com/706392): Fix password reuse detection for Android.