Plumb Task Scheduler Histograms to the Task Scheduler Internals Page

base/task_scheduler/scheduler_worker_pool_impl.h

 // Copyright 2016 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.
 
 #ifndef BASE_TASK_SCHEDULER_SCHEDULER_WORKER_POOL_IMPL_H_
 #define BASE_TASK_SCHEDULER_SCHEDULER_WORKER_POOL_IMPL_H_
 
 #include <stddef.h>
 
 #include <memory>
@@ skipping 80 matching lines @@
                             scoped_refptr<Sequence> sequence,
                             SchedulerWorker* worker) override;
   void PostTaskWithSequenceNow(std::unique_ptr<Task> task,
                                scoped_refptr<Sequence> sequence,
                                SchedulerWorker* worker) override;
 
   const HistogramBase* num_tasks_between_waits_histogram_for_testing() const {
     return num_tasks_between_waits_histogram_;
   }
 
+  void ReportHistograms(std::vector<const HistogramBase*>* histograms) const;

[gab, 2016/10/18 14:28:48]

This isn't "reporting" histograms (i.e. sending to to UMA), it's "getting"
histograms IMO and since TaskScheduler already uses GetHistograms, it seems
natural to use this here as well.

[robliao, 2016/10/18 20:40:25]

Done.

+
  private:
   class SchedulerSingleThreadTaskRunner;
   class SchedulerWorkerDelegateImpl;
 
   SchedulerWorkerPoolImpl(StringPiece name,
                           SchedulerWorkerPoolParams::IORestriction
                               io_restriction,
                           const TimeDelta& suggested_reclaim_time,
                           TaskTracker* task_tracker,
                           DelayedTaskManager* delayed_task_manager);
@@ skipping 85 matching lines @@
   TaskTracker* const task_tracker_;
   DelayedTaskManager* const delayed_task_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(SchedulerWorkerPoolImpl);
 };
 
 }  // namespace internal
 }  // namespace base
 
 #endif  // BASE_TASK_SCHEDULER_SCHEDULER_WORKER_POOL_IMPL_H_