Removing file_thread_task_runner parameter from SerialIoHandler

tools/battor_agent/battor_agent_bin.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.
 
 // This file provides a thin binary wrapper around the BattOr Agent
 // library. This binary wrapper provides a means for non-C++ tracing
 // controllers, such as Telemetry and Android Systrace, to issue high-level
 // tracing commands to the BattOr through an interactive shell.
 //
 // Example usage of how an external trace controller might use this binary:
@@ skipping 292 matching lines @@
   // and deleting it, MUST happen on the IO thread.
   void CreateAgent(
       const std::string& path,
       scoped_refptr<base::SingleThreadTaskRunner> ui_thread_task_runner,
       base::WaitableEvent* done) {
     // In Chrome, we already have a file thread running. Because the Chrome
     // serial library relies on having it available, we have to spin up our own.
     if (!file_thread_.Start())
       ExitFromThreadStartFailure(kFileThreadName);
 
-    agent_.reset(new BattOrAgent(path, this, file_thread_.task_runner(),
-                                 ui_thread_task_runner));
+    agent_.reset(new BattOrAgent(path, this, ui_thread_task_runner));
     done->Signal();
   }
 
   // Postable task for deleting the BattOrAgent. See the comment for
   // CreateAgent() above regarding why this is necessary.
   void DeleteAgent(base::WaitableEvent* done) {
     agent_.reset();
     done->Signal();
   }
 
  private:
   // NOTE: ui_thread_message_loop_ must appear before ui_thread_run_loop_ here
   // because ui_thread_run_loop_ checks for the current MessageLoop during
   // initialization.
   base::MessageLoopForUI ui_thread_message_loop_;
   base::RunLoop ui_thread_run_loop_;
 
   // Threads needed for serial communication.
   base::Thread io_thread_;
   base::Thread file_thread_;

[Reilly Grant (use Gerrit), 2017/06/07 15:48:56]

This field should be replaced with a call to
base::TaskScheduler::CreateAndStartWithDefaultParams("battor_agent") before
calling bin.Run() in main().

[sujith, 2017/06/08 06:36:38]

Done.

 
   // The agent capable of asynchronously communicating with the BattOr.
   std::unique_ptr<BattOrAgent> agent_;
 
   std::string trace_output_file_;
 };
 
 }  // namespace battor
 
 int main(int argc, char* argv[]) {
   base::AtExitManager exit_manager;
   base::CommandLine::Init(argc, argv);
   battor::BattOrAgentBin bin;
   return bin.Run(argc, argv);
 }