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

chrome/test/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/chrome_build.gni")
 import("//build/config/compiler/compiler.gni")
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//chrome/common/features.gni")
@@ skipping 1094 matching lines @@
     # implemented. See crbug.com/611756
   ]
 
   configs += [ "//build/config:precompiled_headers" ]
 
   deps = [
     ":browser_tests_runner",
     ":repack_pseudo_locales",
     ":test_support",
     "//base",
+    "//chrome/browser:browser",

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

Hmm, why did you add this dependency?  It's already included on line 1613.

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

Done.

+    "//components/safe_browsing_db:v4_feature_list",

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

Would this be appropriate to move down into the desktop-only section, or is it
needed for Android as well?

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

Android does not need this. Moved to desktop-only.

     "//components/spellcheck:build_features",
     "//components/sync:test_support_model",
     "//extensions/features",
     "//media:media_features",
     "//ppapi/features",
     "//printing/features",
     "//rlz/features",
     "//third_party/WebKit/public:features",
   ]
 
@@ skipping 4026 matching lines @@
 }
 
 if (is_win) {
   loadable_module("conflicts_dll") {
     testonly = true
     sources = [
       "conflicts/conflicts_dll.cc",
     ]
   }
 }