Remove the |max_threads| argument from CreateAndSetSimpleTaskScheduler().

base/task_scheduler/task_scheduler.h

Index: base/task_scheduler/task_scheduler.h
diff --git a/base/task_scheduler/task_scheduler.h b/base/task_scheduler/task_scheduler.h
index da2cb79504e9c2472758abcfce4f5ddd51246686..da5df13799fcb49c820b2554e74dae7ba33a546a 100644
--- a/base/task_scheduler/task_scheduler.h
+++ b/base/task_scheduler/task_scheduler.h
@@ -6,6 +6,7 @@
 #define BASE_TASK_SCHEDULER_TASK_SCHEDULER_H_
 
 #include <memory>
+#include <string>
 #include <vector>
 
 #include "base/base_export.h"
@@ -107,10 +108,10 @@ class BASE_EXPORT TaskScheduler {
   // synchronization is required to use the post_task.h API after registering a
   // new TaskScheduler.
 
-  // Creates and sets a task scheduler with one worker pool that can have up to
-  // |max_threads| threads. CHECKs on failure. For tests, prefer
+  // Creates and sets a task scheduler using default params. |name| will be used
+  // to label threads and histograms. CHECKs on failure. For tests, prefer

[robliao, 2017/03/13 23:56:05]

Add guidance on choosing a name (e.g. it should identify your component).

[fdoray, 2017/03/14 15:08:22]

Done.

   // base::test::ScopedTaskScheduler (ensures isolation).
-  static void CreateAndSetSimpleTaskScheduler(int max_threads);
+  static void CreateAndSetSimpleTaskScheduler(const std::string& name);

[robliao, 2017/03/13 23:56:05]

Note this change in the CL description.

[fdoray, 2017/03/14 15:08:22]

Done.

 
   // Creates and sets a task scheduler with custom worker pools. CHECKs on
   // failure. |worker_pool_params_vector| describes the worker pools to create.