Plumb Task Scheduler Histograms to the Task Scheduler Internals Page

base/task_scheduler/scheduler_worker_pool_impl_unittest.cc

 // 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.
 
 #include "base/task_scheduler/scheduler_worker_pool_impl.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <unordered_set>
@@ skipping 615 matching lines @@
   event.Signal();
   worker_pool_->WaitForAllWorkersIdleForTesting();
 
   // Wake up the SchedulerWorker that just became idle by posting a task and
   // wait until it becomes idle again. The SchedulerWorker should record the
   // TaskScheduler.NumTasksBetweenWaits.* histogram on wake up.
   task_runner->PostTask(FROM_HERE, Bind(&DoNothing));
   worker_pool_->WaitForAllWorkersIdleForTesting();
 
   // Verify that counts were recorded to the histogram as expected.
-  EXPECT_EQ(0, worker_pool_->num_tasks_between_waits_histogram_for_testing()
+  EXPECT_EQ(0, worker_pool_->num_tasks_between_waits_histogram()
                    ->SnapshotSamples()
                    ->GetCount(0));
-  EXPECT_EQ(1, worker_pool_->num_tasks_between_waits_histogram_for_testing()
+  EXPECT_EQ(1, worker_pool_->num_tasks_between_waits_histogram()
                    ->SnapshotSamples()
                    ->GetCount(3));
-  EXPECT_EQ(0, worker_pool_->num_tasks_between_waits_histogram_for_testing()
+  EXPECT_EQ(0, worker_pool_->num_tasks_between_waits_histogram()
                    ->SnapshotSamples()
                    ->GetCount(10));
 }
 
 namespace {
 
 void SignalAndWaitEvent(WaitableEvent* signal_event,
                         WaitableEvent* wait_event) {
   signal_event->Signal();
   wait_event->Wait();
@@ skipping 43 matching lines @@
   }
   for (const auto& task_started_event : task_started_events)
     task_started_event->Wait();
 
   // Verify that counts were recorded to the histogram as expected.
   // - The "0" bucket has a count of at least 1 because the SchedulerWorker on
   //   top of the idle stack isn't allowed to detach when its sleep timeout
   //   expires. Instead, it waits on its WaitableEvent again without running a
   //   task. The count may be higher than 1 because of spurious wake ups before
   //   the sleep timeout expires.
-  EXPECT_GE(worker_pool_->num_tasks_between_waits_histogram_for_testing()
+  EXPECT_GE(worker_pool_->num_tasks_between_waits_histogram()
                 ->SnapshotSamples()
                 ->GetCount(0),
             1);
   // - The "1" bucket has a count of |kNumWorkersInWorkerPool| because each
   //   SchedulerWorker ran a task before waiting on its WaitableEvent at the
   //   beginning of the test.
   EXPECT_EQ(static_cast<int>(kNumWorkersInWorkerPool),
-            worker_pool_->num_tasks_between_waits_histogram_for_testing()
+            worker_pool_->num_tasks_between_waits_histogram()
                 ->SnapshotSamples()
                 ->GetCount(1));
-  EXPECT_EQ(0, worker_pool_->num_tasks_between_waits_histogram_for_testing()
+  EXPECT_EQ(0, worker_pool_->num_tasks_between_waits_histogram()
                    ->SnapshotSamples()
                    ->GetCount(10));
 
   tasks_can_exit_event.Signal();
   worker_pool_->WaitForAllWorkersIdleForTesting();
   worker_pool_->DisallowWorkerDetachmentForTesting();
 }
 
 }  // namespace internal
 }  // namespace base