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

chrome/browser/chromeos/extensions/input_method_apitest_chromeos.cc

Index: chrome/browser/chromeos/extensions/input_method_apitest_chromeos.cc
diff --git a/chrome/browser/chromeos/extensions/input_method_apitest_chromeos.cc b/chrome/browser/chromeos/extensions/input_method_apitest_chromeos.cc
index f5cfe23c44afc5f300e8f025e0cd9b2c0d7ed990..0d1ff87ce57a1f9010f02a0a2e43b057c1e2fe05 100644
--- a/chrome/browser/chromeos/extensions/input_method_apitest_chromeos.cc
+++ b/chrome/browser/chromeos/extensions/input_method_apitest_chromeos.cc
@@ -8,6 +8,7 @@
 
 #include "base/command_line.h"
 #include "base/memory/ref_counted.h"
+#include "base/test/scoped_feature_list.h"
 #include "chrome/browser/chromeos/extensions/input_method_event_router.h"
 #include "chrome/common/chrome_features.h"
 #include "chrome/common/pref_names.h"
@@ -76,14 +77,17 @@ class TestListener : public content::NotificationObserver {
 };
 
 class ExtensionInputMethodApiTest : public ExtensionApiTest {
+ public:
   void SetUpCommandLine(base::CommandLine* command_line) override {
     ExtensionApiTest::SetUpCommandLine(command_line);
     command_line->AppendSwitchASCII(
         extensions::switches::kWhitelistedExtensionID,
         "ilanclmaeigfpnmdlgelmhkpkegdioip");
-    command_line->AppendSwitchASCII(switches::kEnableFeatures,
-                                    features::kOptInImeMenu.name);
+    scoped_feature_list_.InitAndEnableFeature(features::kOptInImeMenu);

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

Does this need to be done as part of SetUpCommandLine?  Could it happen during
SetUp() instead?  SetUpCommandLine() is a much more specialized method, and I'd
prefer to minimize the amount of code in it, for clarity.

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

Move all calls to SetUp

   }
+
+ private:
+  base::test::ScopedFeatureList scoped_feature_list_;
 };
 
 }  // namespace