Remove SendOnFocusPing and SendPasswordReusePing flags.

The flags were used for launches on Android in M79. They were set to
enable by default in M81. They will be removed in this CL in M82.

Bug: 995926
Change-Id: I8035064bafd38e3eb239c345e5cde0c5074a6402
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095703Reviewed-by: Daniel Rubery <drubery@chromium.org>
Commit-Queue: Xinghui Lu <xinghuilu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748559}

components/safe_browsing/content/password_protection/password_protection_service.cc

@@ -102,9 +102,6 @@ void PasswordProtectionService::MaybeStartPasswordFieldOnFocusRequest(
     const std::string& hosted_domain) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   RequestOutcome reason;
-  if (!base::FeatureList::IsEnabled(safe_browsing::kSendOnFocusPing)) {
-    return;
-  }
   if (CanSendPing(LoginReputationClientRequest::UNFAMILIAR_LOGIN_PAGE,
                   main_frame_url,
                   GetPasswordProtectionReusedPasswordAccountType(
@@ -129,9 +126,6 @@ void PasswordProtectionService::MaybeStartProtectedPasswordEntryRequest(
     const std::vector<std::string>& matching_domains,
     bool password_field_exists) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
-  if (!base::FeatureList::IsEnabled(safe_browsing::kSendPasswordReusePing)) {
-    return;
-  }
   ReusedPasswordAccountType reused_password_account_type =
       GetPasswordProtectionReusedPasswordAccountType(password_type, username);
   RequestOutcome reason;

components/safe_browsing/core/features.cc

@@ -103,16 +103,6 @@ const base::Feature kSafeBrowsingAvailableOnIOS{
 const base::Feature kSafeBrowsingSeparateNetworkContexts{
     "SafeBrowsingSeparateNetworkContexts", base::FEATURE_DISABLED_BY_DEFAULT};
 
-const base::Feature kSendOnFocusPing {
-  "SafeBrowsingSendOnFocusPing",
-      base::FEATURE_ENABLED_BY_DEFAULT
-};
-
-const base::Feature kSendPasswordReusePing {
-  "SafeBrowsingSendPasswordReusePing",
-      base::FEATURE_ENABLED_BY_DEFAULT
-};
-
 constexpr base::FeatureParam<bool> kShouldFillOldPhishGuardProto{
     &kPasswordProtectionForSignedInUsers, "DeprecateOldProto", false};
 
@@ -154,8 +144,6 @@ constexpr struct {
     {&kRealTimeUrlLookupEnabledWithToken, true},
     {&kSafeBrowsingAvailableOnIOS, true},
     {&kSafeBrowsingSeparateNetworkContexts, true},
-    {&kSendOnFocusPing, true},
-    {&kSendPasswordReusePing, true},
     {&kSuspiciousSiteTriggerQuotaFeature, true},
     {&kThreatDomDetailsTagAndAttributeFeature, false},
     {&kTriggerThrottlerDailyQuotaFeature, false},

components/safe_browsing/core/features.h

@@ -82,12 +82,6 @@ extern const base::Feature kSafeBrowsingAvailableOnIOS;
 // profile.
 extern const base::Feature kSafeBrowsingSeparateNetworkContexts;
 
-// Controls whether Chrome sends on focus ping.
-extern const base::Feature kSendOnFocusPing;
-
-// Controls whether Chrome sends password reuse ping.
-extern const base::Feature kSendPasswordReusePing;
-
 // Controls the daily quota for the suspicious site trigger.
 extern const base::Feature kSuspiciousSiteTriggerQuotaFeature;