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

content/browser/service_worker/service_worker_browsertest.cc

 // 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.
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <map>
 #include <memory>
 #include <utility>
@@ skipping 1320 matching lines @@
 
  private:
   bool data_saver_enabled_;
 };
 
 class ServiceWorkerVersionOffMainThreadFetchTest
     : public ServiceWorkerVersionBrowserTest {
  public:
   ~ServiceWorkerVersionOffMainThreadFetchTest() override {}
   void SetUpCommandLine(base::CommandLine* command_line) override {
-    command_line->AppendSwitchASCII(switches::kEnableFeatures,
-                                    features::kOffMainThreadFetch.name);
+    scoped_feature_list_.InitAndEnableFeature(features::kOffMainThreadFetch);
   }
+
+ private:
+  base::test::ScopedFeatureList scoped_feature_list_;
 };
 
 IN_PROC_BROWSER_TEST_F(ServiceWorkerVersionBrowserTest, FetchWithSaveData) {
   embedded_test_server()->RegisterRequestHandler(
       base::Bind(&VerifySaveDataHeaderInRequest));
   StartServerAndNavigateToSetup();
   MockContentBrowserClient content_browser_client;
   content_browser_client.set_data_saver_enabled(true);
   ContentBrowserClient* old_client =
       SetBrowserClientForTesting(&content_browser_client);
@@ skipping 191 matching lines @@
   using self = ServiceWorkerNavigationPreloadTest;
 
   ~ServiceWorkerNavigationPreloadTest() override {}
 
   void SetUpOnMainThread() override {
     host_resolver()->AddRule("*", "127.0.0.1");
     ServiceWorkerBrowserTest::SetUpOnMainThread();
   }
 
   void SetUpCommandLine(base::CommandLine* command_line) override {
-    command_line->AppendSwitchASCII(
-        switches::kEnableFeatures,
-        features::kServiceWorkerNavigationPreload.name);
+    scoped_feature_list_.InitAndEnableFeature(
+        features::kServiceWorkerNavigationPreload);
   }
 
  protected:
   static const std::string kNavigationPreloadHeaderName;
   static const std::string kEnableNavigationPreloadScript;
   static const std::string kPreloadResponseTestScript;
 
   static bool HasNavigationPreloadHeader(
       const net::test_server::HttpRequest& request) {
     return request.headers.find(kNavigationPreloadHeaderName) !=
@@ skipping 87 matching lines @@
         base::ASCIIToUTF16("document.body.textContent;"),
         base::Bind(&StoreString, &text_content, run_loop.QuitClosure()));
     run_loop.Run();
     return text_content;
   }
 
   std::map<std::string, std::vector<net::test_server::HttpRequest>>
       request_log_;
 
  private:
+  base::test::ScopedFeatureList scoped_feature_list_;
   class CustomResponse : public net::test_server::HttpResponse {
    public:
     CustomResponse(const std::string& response) : response_(response) {}
     ~CustomResponse() override {}
 
     void SendResponse(
         const net::test_server::SendBytesCallback& send,
         const net::test_server::SendCompleteCallback& done) override {
       send.Run(response_, done);
     }
@@ skipping 1355 matching lines @@
   // effect in CanSuspendRenderer().
   shell()->web_contents()->WasHidden();
   EXPECT_TRUE(rph->IsProcessBackgrounded());
 
   // The process which has service worker thread shouldn't be suspended.
   EXPECT_FALSE(memory_coordinator->CanSuspendRenderer(render_process_id));
 }
 #endif
 
 }  // namespace content