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

Side by Side Diff: components/scheduler/renderer/renderer_scheduler_impl.h

Issue 2184023002: Report the duration of sampled tasks with histogram. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 months 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_SCHEDULER_RENDERER_RENDERER_SCHEDULER_IMPL_H_ 5 #ifndef COMPONENTS_SCHEDULER_RENDERER_RENDERER_SCHEDULER_IMPL_H_
6 #define COMPONENTS_SCHEDULER_RENDERER_RENDERER_SCHEDULER_IMPL_H_ 6 #define COMPONENTS_SCHEDULER_RENDERER_RENDERER_SCHEDULER_IMPL_H_
7 7
8 #include "base/atomicops.h" 8 #include "base/atomicops.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "base/synchronization/lock.h" 10 #include "base/synchronization/lock.h"
11 #include "components/scheduler/base/long_task_tracker.h" 11 #include "components/scheduler/base/long_task_tracker.h"
12 #include "components/scheduler/base/pollable_thread_safe_flag.h" 12 #include "components/scheduler/base/pollable_thread_safe_flag.h"
13 #include "components/scheduler/base/queueing_time_estimator.h" 13 #include "components/scheduler/base/queueing_time_estimator.h"
14 #include "components/scheduler/base/task_time_tracker.h" 14 #include "components/scheduler/base/task_time_tracker.h"
15 #include "components/scheduler/child/idle_helper.h" 15 #include "components/scheduler/child/idle_helper.h"
16 #include "components/scheduler/child/scheduler_helper.h" 16 #include "components/scheduler/child/scheduler_helper.h"
17 #include "components/scheduler/renderer/deadline_task_runner.h" 17 #include "components/scheduler/renderer/deadline_task_runner.h"
18 #include "components/scheduler/renderer/idle_time_estimator.h" 18 #include "components/scheduler/renderer/idle_time_estimator.h"
19 #include "components/scheduler/renderer/render_widget_signals.h" 19 #include "components/scheduler/renderer/render_widget_signals.h"
20 #include "components/scheduler/renderer/renderer_scheduler.h" 20 #include "components/scheduler/renderer/renderer_scheduler.h"
21 #include "components/scheduler/renderer/renderer_task_duration.h"
21 #include "components/scheduler/renderer/task_cost_estimator.h" 22 #include "components/scheduler/renderer/task_cost_estimator.h"
22 #include "components/scheduler/renderer/throttling_helper.h" 23 #include "components/scheduler/renderer/throttling_helper.h"
23 #include "components/scheduler/renderer/user_model.h" 24 #include "components/scheduler/renderer/user_model.h"
24 #include "components/scheduler/renderer/web_view_scheduler_impl.h" 25 #include "components/scheduler/renderer/web_view_scheduler_impl.h"
25 #include "components/scheduler/scheduler_export.h" 26 #include "components/scheduler/scheduler_export.h"
26 27
27 namespace base { 28 namespace base {
28 namespace trace_event { 29 namespace trace_event {
29 class ConvertableToTraceFormat; 30 class ConvertableToTraceFormat;
30 } 31 }
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
128 129
129 // SchedulerHelper::Observer implementation: 130 // SchedulerHelper::Observer implementation:
130 void OnUnregisterTaskQueue(const scoped_refptr<TaskQueue>& queue) override; 131 void OnUnregisterTaskQueue(const scoped_refptr<TaskQueue>& queue) override;
131 void OnTriedToExecuteBlockedTask(const TaskQueue& queue, 132 void OnTriedToExecuteBlockedTask(const TaskQueue& queue,
132 const base::PendingTask& task) override; 133 const base::PendingTask& task) override;
133 134
134 // TaskTimeTracker implementation: 135 // TaskTimeTracker implementation:
135 void ReportTaskTime(base::TimeTicks start_time, 136 void ReportTaskTime(base::TimeTicks start_time,
136 base::TimeTicks end_time) override; 137 base::TimeTicks end_time) override;
137 138
139 // Sample Tasks and Report in histogrtam":
140 void ReportSampleDuration(base::TimeTicks start_time,
141 base::TimeTicks end_time);
142
majidvp 2016/07/27 13:50:59 I think this is is not longer needed.
sunyunjia 2016/07/27 17:28:42 Done.
138 // QueueingTimeEstimator::Client implementation: 143 // QueueingTimeEstimator::Client implementation:
139 void OnQueueingTimeForWindowEstimated(base::TimeDelta queueing_time) override; 144 void OnQueueingTimeForWindowEstimated(base::TimeDelta queueing_time) override;
140 145
141 // Returns a task runner where tasks run at the highest possible priority. 146 // Returns a task runner where tasks run at the highest possible priority.
142 scoped_refptr<TaskQueue> ControlTaskRunner(); 147 scoped_refptr<TaskQueue> ControlTaskRunner();
143 148
144 void RegisterTimeDomain(TimeDomain* time_domain); 149 void RegisterTimeDomain(TimeDomain* time_domain);
145 void UnregisterTimeDomain(TimeDomain* time_domain); 150 void UnregisterTimeDomain(TimeDomain* time_domain);
146 151
147 void SetExpensiveTaskBlockingAllowed(bool allowed); 152 void SetExpensiveTaskBlockingAllowed(bool allowed);
(...skipping 315 matching lines...) Expand 10 before | Expand all | Expand 10 after
463 } 468 }
464 469
465 // Don't access compositor_thread_only_, instead use CompositorThreadOnly(). 470 // Don't access compositor_thread_only_, instead use CompositorThreadOnly().
466 CompositorThreadOnly compositor_thread_only_; 471 CompositorThreadOnly compositor_thread_only_;
467 CompositorThreadOnly& CompositorThreadOnly() { 472 CompositorThreadOnly& CompositorThreadOnly() {
468 compositor_thread_only_.CheckOnValidThread(); 473 compositor_thread_only_.CheckOnValidThread();
469 return compositor_thread_only_; 474 return compositor_thread_only_;
470 } 475 }
471 476
472 PollableThreadSafeFlag policy_may_need_update_; 477 PollableThreadSafeFlag policy_may_need_update_;
478
479 RendererTaskDuration renderer_task_duration_;
480
473 base::WeakPtrFactory<RendererSchedulerImpl> weak_factory_; 481 base::WeakPtrFactory<RendererSchedulerImpl> weak_factory_;
474 482
475 DISALLOW_COPY_AND_ASSIGN(RendererSchedulerImpl); 483 DISALLOW_COPY_AND_ASSIGN(RendererSchedulerImpl);
476 }; 484 };
477 485
478 } // namespace scheduler 486 } // namespace scheduler
479 487
480 #endif // COMPONENTS_SCHEDULER_RENDERER_RENDERER_SCHEDULER_IMPL_H_ 488 #endif // COMPONENTS_SCHEDULER_RENDERER_RENDERER_SCHEDULER_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698