commit | 52fa3aed17a23a6d1dc7459879c9fb2abc689bc8 | [log] [tgz] |
---|---|---|
author | Gabriel Charette <[email protected]> | Mon Apr 15 21:44:37 2019 |
committer | Commit Bot <[email protected]> | Mon Apr 15 21:44:37 2019 |
tree | bc76616228684b48bb8b956197951af52c908754 | |
parent | 95d4d3045a8f9302d2be3fcc73274104305e1d6c [diff] |
[base] Rename TaskScheduler to ThreadPool Reason: with the advent of other scheduling primitives in //base (i.e. SequenceManager), TaskScheduler was no longer the only component responsible for scheduling tasks. We will from now on refer to the whole of //base/task as the "task scheduling infrastructure". There are other types named "TaskScheduler" outside of base:: so s/TaskScheduler/ThreadPool/ across the codebase wasn't possible. Instead, this CL did: 1) base/task/task_scheduler => base/task/thread_pool (catches all files with includes) 1.1) Careful manual search to add files without includes (e.g. missing IWYU, docs, etc.) 2) TaskScheduler => ThreadPool in all files affected by (1) 3) task_scheduler => thread_pool in all files affected by (1) 4) "task scheduler" => "thread pool" in all files affected by (1) 4) Move task_scheduler_util like headers in //content //components and //ios Also: * Renamed UMA metrics from TaskScheduler.* to ThreadPool.* and dropped "Pool" from worker pool name suffixes. * Renamed TaskScheduler*Worker thread names to ThreadPool*Worker * In base/android: NativeTaskScheduler => NativeScheduler as it was referring to the whole of base/task. TaskSchedulerTest.java => NativePostTaskTest.java (former DNE) * Intentionally ignoring IWYU violations in this already too large CL. In follow-up: * Rename other types as well: SchedulerWorker => WorkerThread SchedulerWorkerPool* => WorkerThreadGroup* Bug: 951388 Change-Id: I5bc2688b593c7682ef7e56d6b228539970ba107e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1561552 Commit-Queue: Gabriel Charette <[email protected]> Auto-Submit: Gabriel Charette <[email protected]> Reviewed-by: John Abd-El-Malek <[email protected]> Reviewed-by: François Doray <[email protected]> Reviewed-by: Ilya Sherman <[email protected]> Reviewed-by: Richard Coles <[email protected]> Reviewed-by: Joe Mason <[email protected]> Reviewed-by: Etienne Pierre-Doray <[email protected]> Cr-Commit-Position: refs/heads/master@{#650997}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .