Sending reporter logs doesn't depend on a feature anymore.

chrome/browser/safe_browsing/srt_fetcher_browsertest_win.cc

 // Copyright 2015 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 "chrome/browser/safe_browsing/srt_fetcher_win.h"
 
 #include <iterator>
 #include <memory>
 #include <set>
 
@@ skipping 527 matching lines @@
     // If we try again before the reporting period is up, it should not do
     // anything.
     TestReporterLaunchCycle({});
 
     // After enough time has passed, should try the queue again.
     SetDaysSinceLastReport(kDaysBetweenSuccessfulSwReporterRuns);
     TestReporterLaunchCycle({path1});
   }
 }
 
-IN_PROC_BROWSER_TEST_F(SRTFetcherTest, ReporterLogging_FeatureDisabled) {
-  exit_code_to_report_ = kSwReporterNothingFound;
-  base::test::ScopedFeatureList scoped_feature_list;
-  scoped_feature_list.InitAndDisableFeature(
-      kSwReporterExtendedSafeBrowsingFeature);
-  RunReporter();
-  TestReporterLaunchCycle({base::FilePath()});
-  ExpectLoggingSwitches({/*expect no switches*/});
-  ExpectLastTimeSentReportNotSet();
-}
-
 IN_PROC_BROWSER_TEST_F(SRTFetcherTest, ReporterLogging_NoSBExtendedReporting) {
   exit_code_to_report_ = kSwReporterNothingFound;
   base::test::ScopedFeatureList scoped_feature_list;
-  scoped_feature_list.InitAndEnableFeature(
-      kSwReporterExtendedSafeBrowsingFeature);
   RunReporter();
   TestReporterLaunchCycle({base::FilePath()});
   ExpectLoggingSwitches({/*expect no switches*/});
   ExpectLastTimeSentReportNotSet();
 }
 
 IN_PROC_BROWSER_TEST_F(SRTFetcherTest, ReporterLogging_EnabledFirstRun) {
   exit_code_to_report_ = kSwReporterNothingFound;
   base::test::ScopedFeatureList scoped_feature_list;
-  scoped_feature_list.InitAndEnableFeature(
-      kSwReporterExtendedSafeBrowsingFeature);
   EnableSBExtendedReporting();
   // Note: don't set last time sent logs in the local state.
   // SBER is enabled and there is no record in the local state of the last time
   // logs have been sent, so we should send logs in this run.
   RunReporter();
   TestReporterLaunchCycle({base::FilePath()});
   ExpectLoggingSwitches(std::set<std::string>(std::begin(kExpectedSwitches),
                                               std::end(kExpectedSwitches)));
   ExpectLastReportSentInTheLastHour();
 }
 
 IN_PROC_BROWSER_TEST_F(SRTFetcherTest, ReporterLogging_EnabledNoRecentLogging) {
   exit_code_to_report_ = kSwReporterNothingFound;
   base::test::ScopedFeatureList scoped_feature_list;
-  scoped_feature_list.InitAndEnableFeature(
-      kSwReporterExtendedSafeBrowsingFeature);
   // SBER is enabled and last time logs were sent was more than
   // |kDaysBetweenReporterLogsSent| day ago, so we should send logs in this run.
   EnableSBExtendedReporting();
   SetLastTimeSentReport(kDaysBetweenReporterLogsSent + 3);
   RunReporter();
   TestReporterLaunchCycle({base::FilePath()});
   ExpectLoggingSwitches(std::set<std::string>(std::begin(kExpectedSwitches),
                                               std::end(kExpectedSwitches)));
   ExpectLastReportSentInTheLastHour();
 }
 
 IN_PROC_BROWSER_TEST_F(SRTFetcherTest, ReporterLogging_EnabledRecentlyLogged) {
   exit_code_to_report_ = kSwReporterNothingFound;
   base::test::ScopedFeatureList scoped_feature_list;
-  scoped_feature_list.InitAndEnableFeature(
-      kSwReporterExtendedSafeBrowsingFeature);
   // SBER is enabled, but logs have been sent less than
   // |kDaysBetweenReporterLogsSent| day ago, so we shouldn't send any logs in
   // this run.
   EnableSBExtendedReporting();
   SetLastTimeSentReport(kDaysBetweenReporterLogsSent - 1);
   int64_t last_time_sent_logs = GetLastTimeSentReport();
   RunReporter();
   TestReporterLaunchCycle({base::FilePath()});
   ExpectLoggingSwitches(std::set<std::string>{/*expect no switches*/});
   EXPECT_EQ(last_time_sent_logs, GetLastTimeSentReport());
 }
 
 }  // namespace safe_browsing