Show modal warning for saved password for low rep.

Previously, we filtered out low reputation sites from
the modal warnings shown for phished svaed passwords.
However, low reputation and phished sites should be both
shown as warnings.

Bug: 1026000
Change-Id: I425198650c387bdccbb72769c45d054f91075e57
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1921654Reviewed-by: Varun Khaneja <vakh@chromium.org>
Commit-Queue: Bettina Dea <bdea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#716369}

components/safe_browsing/password_protection/password_protection_service.cc

@@ -172,13 +172,6 @@ bool PasswordProtectionService::ShouldShowModalWarning(
     return false;
   }
 
-  if (password_type.account_type() ==
-          ReusedPasswordAccountType::SAVED_PASSWORD &&
-      base::FeatureList::IsEnabled(
-          safe_browsing::kPasswordProtectionForSavedPasswords)) {
-    return verdict_type == LoginReputationClientResponse::PHISHING;
-  }
-
   return (verdict_type == LoginReputationClientResponse::PHISHING ||
           verdict_type == LoginReputationClientResponse::LOW_REPUTATION) &&
          IsWarningEnabled(password_type);

components/safe_browsing/password_protection/password_protection_service_unittest.cc

@@ -1154,6 +1154,10 @@ TEST_P(PasswordProtectionServiceTest, VerifyShouldShowModalWarning) {
     EXPECT_TRUE(password_protection_service_->ShouldShowModalWarning(
         LoginReputationClientRequest::PASSWORD_REUSE_EVENT,
         reused_password_account_type, LoginReputationClientResponse::PHISHING));
+    EXPECT_TRUE(password_protection_service_->ShouldShowModalWarning(
+        LoginReputationClientRequest::PASSWORD_REUSE_EVENT,
+        reused_password_account_type,
+        LoginReputationClientResponse::LOW_REPUTATION));
   }
 
   {