Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(698)

Unified Diff: base/task_scheduler/scheduler_worker_pool_impl.h

Issue 2600023002: Remove worker pool names from TaskScheduler.TaskLatency.* histograms. (Closed)
Patch Set: self-review Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: base/task_scheduler/scheduler_worker_pool_impl.h
diff --git a/base/task_scheduler/scheduler_worker_pool_impl.h b/base/task_scheduler/scheduler_worker_pool_impl.h
index b2dc68a4f12c96c79e8e27c41a4187b3a1c6b421..5d25ce1a6908e60f9a959c0df833df52cd430f70 100644
--- a/base/task_scheduler/scheduler_worker_pool_impl.h
+++ b/base/task_scheduler/scheduler_worker_pool_impl.h
@@ -11,7 +11,6 @@
#include <string>
#include <vector>
-#include "base/atomicops.h"
#include "base/base_export.h"
#include "base/callback.h"
#include "base/logging.h"
@@ -28,12 +27,12 @@
#include "base/task_scheduler/scheduler_worker_stack.h"
#include "base/task_scheduler/sequence.h"
#include "base/task_scheduler/task.h"
-#include "base/task_scheduler/task_traits.h"
#include "base/threading/platform_thread.h"
namespace base {
class HistogramBase;
+class TaskTraits;
class TimeDelta;
namespace internal {
@@ -204,14 +203,6 @@ class BASE_EXPORT SchedulerWorkerPoolImpl : public SchedulerWorkerPool {
// Intentionally leaked.
HistogramBase* const num_tasks_between_waits_histogram_;
- // TaskScheduler.TaskLatency.[worker pool name].[task priority] histograms.
- // Indexed by task priority. Histograms are allocated on demand to reduce
- // memory usage (some task priorities might never run in this
- // SchedulerThreadPoolImpl). Intentionally leaked.
- subtle::AtomicWord
- task_latency_histograms_[static_cast<int>(TaskPriority::HIGHEST) + 1] =
- {};
-
TaskTracker* const task_tracker_;
DelayedTaskManager* const delayed_task_manager_;

Powered by Google App Engine
This is Rietveld 408576698