GTTF: Enable --brave-new-test-launcher by default.

base/test/launcher/unit_test_launcher.cc

 // Copyright 2013 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 "base/test/launcher/unit_test_launcher.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
@@ skipping 14 matching lines @@
 
 namespace base {
 
 namespace {
 
 // This constant controls how many tests are run in a single batch by default.
 const size_t kDefaultTestBatchLimit = 10;
 
 const char kHelpFlag[] = "help";
 
-// Flag to enable the new launcher logic.
-// TODO(phajdan.jr): Remove it, http://crbug.com/236893 .
-const char kBraveNewTestLauncherFlag[] = "brave-new-test-launcher";
-
 // Flag to run all tests in a single process.
 const char kSingleProcessTestsFlag[] = "single-process-tests";
 
 void PrintUsage() {
   fprintf(stdout,
           "Runs tests using the gtest framework, each batch of tests being\n"
           "run in their own process. Supported command-line flags:\n"
           "\n"
           "  --single-process-tests\n"
           "    Runs the tests and the launcher in the same process. Useful\n"
@@ skipping 15 matching lines @@
 // of current process. |test_names| is a vector of test full names
 // (e.g. "A.B"), |output_file| is path to the GTest XML output file.
 CommandLine GetCommandLineForChildGTestProcess(
     const std::vector<std::string>& test_names,
     const base::FilePath& output_file) {
   CommandLine new_cmd_line(*CommandLine::ForCurrentProcess());
 
   new_cmd_line.AppendSwitchPath(switches::kTestLauncherOutput, output_file);
   new_cmd_line.AppendSwitchASCII(kGTestFilterFlag, JoinString(test_names, ":"));
   new_cmd_line.AppendSwitch(kSingleProcessTestsFlag);
-  new_cmd_line.AppendSwitch(kBraveNewTestLauncherFlag);
 
   return new_cmd_line;
 }
 
 class UnitTestLauncherDelegate : public TestLauncherDelegate {
  public:
   explicit UnitTestLauncherDelegate(size_t batch_limit)
       : batch_limit_(batch_limit) {
   }
 
@@ skipping 354 matching lines @@
 
   return true;
 }
 
 int LaunchUnitTestsInternal(int argc,
                             char** argv,
                             const RunTestSuiteCallback& run_test_suite,
                             int default_jobs) {
   CommandLine::Init(argc, argv);
   if (CommandLine::ForCurrentProcess()->HasSwitch(kGTestHelpFlag) ||
-      CommandLine::ForCurrentProcess()->HasSwitch(kSingleProcessTestsFlag) ||
-      !CommandLine::ForCurrentProcess()->HasSwitch(kBraveNewTestLauncherFlag)) {
+      CommandLine::ForCurrentProcess()->HasSwitch(kSingleProcessTestsFlag)) {
     return run_test_suite.Run();
   }
 
   if (CommandLine::ForCurrentProcess()->HasSwitch(kHelpFlag)) {
     PrintUsage();
     return 0;
   }
 
   base::TimeTicks start_time(base::TimeTicks::Now());
 
@@ skipping 34 matching lines @@
       argc, argv, run_test_suite, SysInfo::NumberOfProcessors());
 }
 
 int LaunchUnitTestsSerially(int argc,
                             char** argv,
                             const RunTestSuiteCallback& run_test_suite) {
   return LaunchUnitTestsInternal(argc, argv, run_test_suite, 1);
 }
 
 }  // namespace base