[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 1318 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);
+  void SetUp() override {
+    scoped_feature_list_.InitAndEnableFeature(features::kOffMainThreadFetch);
+    ServiceWorkerVersionBrowserTest::SetUp();
   }
+
+ 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 190 matching lines @@
  public:
   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);
+  void SetUp() override {
+    scoped_feature_list_.InitAndEnableFeature(
+        features::kServiceWorkerNavigationPreload);
+    ServiceWorkerBrowserTest::SetUp();
   }
 
  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 1291 matching lines @@
 IN_PROC_BROWSER_TEST_F(ServiceWorkerDisableWebSecurityTest, UpdateNoCrash) {
   StartServerAndNavigateToSetup();
   const char kPageUrl[] = "/service_worker/disable_web_security_update.html";
   const char kScopeUrl[] = "/service_worker/scope/";
   const char kWorkerUrl[] = "/service_worker/fetch_event_blob.js";
   RegisterServiceWorkerOnCrossOriginServer(kScopeUrl, kWorkerUrl);
   RunTestWithCrossOriginURL(kPageUrl, kScopeUrl);
 }
 
 }  // namespace content