[Cleanup] Move all browsertests to use ScopedFeatureList to modify features

components/safe_browsing_db/v4_feature_list.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_SAFE_BROWSING_DB_V4_FEATURE_LIST_H_
 #define COMPONENTS_SAFE_BROWSING_DB_V4_FEATURE_LIST_H_
 
+#include "base/feature_list.h"
+
+namespace features {
+
+extern const base::Feature kV4OnlyEnabled;
+
+}  // namespace features

[Ilya Sherman, 2017/06/12 21:50:29]

Please define a separate features.h file for this feature (and a corresponding
impl file for the definition.)

[chaopeng, 2017/06/28 19:21:25]

Done.

+
 namespace safe_browsing {
 
 // Exposes methods to check whether a particular feature has been enabled
 // through Finch.
 namespace V4FeatureList {
 
 enum class V4UsageStatus {
   // The V4 database manager is not even instantiated i.e. is diabled. All
   // SafeBrowsing operations use PVer3 code.
   V4_DISABLED,
 
   // The V4 database manager is instantiated, and performs background updates,
   // but all SafeBrowsing verdicts are returned using the PVer3 database.
   V4_INSTANTIATED,
 
   // Only the V4 database manager is instantiated, PVer3 database manager is
   // not. All SafeBrowsing verdicts are returned using PVer4 database.
   V4_ONLY
 };
 
 V4UsageStatus GetV4UsageStatus();
 
 }  // namespace V4FeatureList
 
 }  // namespace safe_browsing
 
 #endif  // COMPONENTS_SAFE_BROWSING_DB_V4_FEATURE_LIST_H_