Remove feature flags for PVer4 -- no longer used

TBR=asvitkine

Bug: 754336
Change-Id: I316e03853095b49dc9da6d6d19fccfd9cefb7bf5
Reviewed-on: https://chromium-review.googlesource.com/773505
Commit-Queue: Varun Khaneja <vakh@chromium.org>
Reviewed-by: Nathan Parker <nparker@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517688}

components/safe_browsing/features.cc

@@ -35,10 +35,6 @@ const base::Feature kGoogleBrandedPhishingWarning{
     "PasswordProtectionGoogleBrandedPhishingWarning",
     base::FEATURE_DISABLED_BY_DEFAULT};
 
-const base::Feature kLocalDatabaseManagerEnabled{
-    "SafeBrowsingV4LocalDatabaseManagerEnabled",
-    base::FEATURE_DISABLED_BY_DEFAULT};
-
 // If enabled, SafeBrowsing URL checks don't defer starting requests or
 // following redirects, no matter on desktop or mobile. Instead they only defer
 // response processing.
@@ -55,9 +51,6 @@ const base::Feature kTriggerThrottlerDailyQuotaFeature{
     "SafeBrowsingTriggerThrottlerDailyQuota",
     base::FEATURE_DISABLED_BY_DEFAULT};
 
-const base::Feature kV4OnlyEnabled{"SafeBrowsingV4OnlyEnabled",
-                                   base::FEATURE_DISABLED_BY_DEFAULT};
-
 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
@@ -71,11 +64,9 @@ constexpr struct {
     {&kAdSamplerTriggerFeature, false},
     {&kGaiaPasswordReuseReporting, true},
     {&kGoogleBrandedPhishingWarning, true},
-    {&kLocalDatabaseManagerEnabled, true},
     {&kParallelUrlCheck, true},
     {&kThreatDomDetailsTagAndAttributeFeature, false},
     {&kTriggerThrottlerDailyQuotaFeature, false},
-    {&kV4OnlyEnabled, true},
 };
 
 // Adds the name and the enabled/disabled status of a given feature.

components/safe_browsing/features.h

@@ -24,7 +24,6 @@ extern const base::Feature kAdSamplerTriggerFeature;
 // Gates logging of GaiaPasswordReuse user events.
 extern const base::Feature kGaiaPasswordReuseReporting;
 extern const base::Feature kGoogleBrandedPhishingWarning;
-extern const base::Feature kLocalDatabaseManagerEnabled;
 extern const base::Feature kParallelUrlCheck;
 
 // Specifies which non-resource HTML Elements to collect based on their tag and
@@ -41,8 +40,6 @@ extern const base::Feature kThreatDomDetailsTagAndAttributeFeature;
 // that trigger is allowed to send per day.
 extern const base::Feature kTriggerThrottlerDailyQuotaFeature;
 
-extern const base::Feature kV4OnlyEnabled;
-
 base::ListValue GetFeatureStatusList();
 
 #endif  // COMPONENTS_SAFE_BROWSING_FEATURES_H_

testing/variations/fieldtrial_testing_config.json

@@ -3218,25 +3218,6 @@
             ]
         }
     ],
-    "SafeBrowsingV4LocalDatabaseManagerEnabled": [
-        {
-            "platforms": [
-                "chromeos",
-                "linux",
-                "mac",
-                "win"
-            ],
-            "experiments": [
-                {
-                    "name": "V4Only2",
-                    "enable_features": [
-                        "SafeBrowsingV4LocalDatabaseManagerEnabled",
-                        "SafeBrowsingV4OnlyEnabled"
-                    ]
-                }
-            ]
-        }
-    ],
     "SearchEnginePromo": [
         {
             "platforms": [