WorkerThread CPU Load UMA

third_party/WebKit/Source/platform/scheduler/child/worker_scheduler_impl.h

 // Copyright 2015 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 THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_CHILD_WORKER_SCHEDULER_IMPL_H_
 #define THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_CHILD_WORKER_SCHEDULER_IMPL_H_
 
 #include "base/macros.h"
+#include "platform/scheduler/base/thread_load_tracker.h"
 #include "platform/scheduler/child/idle_canceled_delayed_task_sweeper.h"
 #include "platform/scheduler/child/idle_helper.h"
 #include "platform/scheduler/child/scheduler_helper.h"
+#include "public/platform/scheduler/base/task_time_observer.h"
 #include "public/platform/scheduler/child/worker_scheduler.h"
 
 namespace blink {
 namespace scheduler {
 
 class SchedulerTqmDelegate;
 
 class BLINK_PLATFORM_EXPORT WorkerSchedulerImpl : public WorkerScheduler,
-                                                  public IdleHelper::Delegate {
+                                                  public IdleHelper::Delegate,
+                                                  TaskTimeObserver {

[nhiroki, 2017/03/23 05:20:04]

No public/protected/private. Is this intended?

[kinuko, 2017/04/03 13:01:20]

Not really, made it public.

  public:
   explicit WorkerSchedulerImpl(
       scoped_refptr<SchedulerTqmDelegate> main_task_runner);
   ~WorkerSchedulerImpl() override;
 
   // WorkerScheduler implementation:
   scoped_refptr<TaskQueue> DefaultTaskRunner() override;
   scoped_refptr<SingleThreadIdleTaskRunner> IdleTaskRunner() override;
   bool CanExceedIdleDeadlineIfRequired() const override;
   bool ShouldYieldForHighPriorityWork() override;
   void AddTaskObserver(base::MessageLoop::TaskObserver* task_observer) override;
   void RemoveTaskObserver(
       base::MessageLoop::TaskObserver* task_observer) override;
   void Init() override;
   void Shutdown() override;
 
+  // TaskTimeObserver implementation:
+  void willProcessTask(TaskQueue* task_queue, double start_time) override;
+  void didProcessTask(TaskQueue* task_queue,
+                      double start_time,
+                      double end_time) override;
+
   SchedulerHelper* GetSchedulerHelperForTesting();
   base::TimeTicks CurrentIdleTaskDeadlineForTesting() const;
 
  protected:
   // IdleHelper::Delegate implementation:
   bool CanEnterLongIdlePeriod(
       base::TimeTicks now,
       base::TimeDelta* next_long_idle_period_delay_out) override;
   void IsNotQuiescent() override {}
   void OnIdlePeriodStarted() override {}
   void OnIdlePeriodEnded() override {}
 
  private:
   void MaybeStartLongIdlePeriod();
 
   SchedulerHelper helper_;
   IdleHelper idle_helper_;
   IdleCanceledDelayedTaskSweeper idle_canceled_delayed_task_sweeper_;
+  ThreadLoadTracker load_tracker_;

[nhiroki, 2017/03/23 05:20:04]

Hm... it looks like this field name doesn't obey the blink's coding standard
(this should be m_loadTracker), I'm ok with keeping this as is for consistency
with others in this file though.

[Sami, 2017/03/24 10:48:10]

This code originally lived in content/ and then moved into Blink. We didn't
change the naming because Blink is planning to switch to Chromium style soon
anyway, so let's keep using Chromium style here.

[nhiroki, 2017/04/04 03:13:06]

I see. Thank you for the clarification

   bool initialized_;
 
   DISALLOW_COPY_AND_ASSIGN(WorkerSchedulerImpl);
 };
 
 }  // namespace scheduler
 }  // namespace blink
 
 #endif  // THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_CHILD_WORKER_SCHEDULER_IMPL_H_