Add a param to the kPasswordProtectionForSignedInUsers experiment.

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

components/safe_browsing/features.cc

@@ -9,6 +9,7 @@
 #include <utility>
 #include <vector>
 #include "base/feature_list.h"
+#include "base/metrics/field_trial_params.h"
 
 #include "base/macros.h"
 #include "base/values.h"
@@ -61,6 +62,9 @@ const base::Feature kUseAPDownloadProtection{"UseAPDownloadProtection",
 const base::Feature kForceUseAPDownloadProtection{
     "ForceUseAPDownloadProtection", base::FEATURE_DISABLED_BY_DEFAULT};
 
+constexpr base::FeatureParam<bool> kShouldFillOldPhishGuardProto{
+    &kPasswordProtectionForSignedInUsers, "DeprecateOldProto", false};
+
 namespace {
 // List of experimental features. Boolean value for each list member should be
 // set to true if the experiment is currently running at a probability other
@@ -106,4 +110,8 @@ base::ListValue GetFeatureStatusList() {
   return param_list;
 }
 
+bool GetShouldFillOldPhishGuardProto() {
+  return kShouldFillOldPhishGuardProto.Get();
+}
+
 }  // namespace safe_browsing

components/safe_browsing/features.h

@@ -66,5 +66,10 @@ extern const base::Feature kForceUseAPDownloadProtection;
 
 base::ListValue GetFeatureStatusList();
 
+// Returns whether or not to stop filling in the SyncAccountType and
+// ReusedPasswordType enums. This is used in the
+// kPasswordProtectionForSignedInUsers experiment.
+bool GetShouldFillOldPhishGuardProto();
+
 }  // namespace safe_browsing
 #endif  // COMPONENTS_SAFE_BROWSING_FEATURES_H_