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

Side by Side Diff: third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.h

Issue 2866613002: EQT: Change Expected Queuing Time from per-second to sliding window (Closed)
Patch Set: uma report per window duration; sliding window include the starting points Created 3 years, 7 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 THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDULER _IMPL_H_ 5 #ifndef THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDULER _IMPL_H_
6 #define THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDULER _IMPL_H_ 6 #define THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDULER _IMPL_H_
7 7
8 #include "base/atomicops.h" 8 #include "base/atomicops.h"
9 #include "base/gtest_prod_util.h" 9 #include "base/gtest_prod_util.h"
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 const base::PendingTask& task) override; 131 const base::PendingTask& task) override;
132 132
133 // TaskTimeObserver implementation: 133 // TaskTimeObserver implementation:
134 void WillProcessTask(TaskQueue* task_queue, double start_time) override; 134 void WillProcessTask(TaskQueue* task_queue, double start_time) override;
135 void DidProcessTask(TaskQueue* task_queue, 135 void DidProcessTask(TaskQueue* task_queue,
136 double start_time, 136 double start_time,
137 double end_time) override; 137 double end_time) override;
138 void OnBeginNestedMessageLoop() override; 138 void OnBeginNestedMessageLoop() override;
139 139
140 // QueueingTimeEstimator::Client implementation: 140 // QueueingTimeEstimator::Client implementation:
141 void OnQueueingTimeForWindowEstimated(base::TimeDelta queueing_time) override; 141 void OnQueueingTimeForWindowEstimated(
142 base::TimeDelta queueing_time,
143 base::TimeTicks window_start_time) override;
142 144
143 scoped_refptr<TaskQueue> DefaultTaskQueue(); 145 scoped_refptr<TaskQueue> DefaultTaskQueue();
144 scoped_refptr<TaskQueue> CompositorTaskQueue(); 146 scoped_refptr<TaskQueue> CompositorTaskQueue();
145 scoped_refptr<TaskQueue> LoadingTaskQueue(); 147 scoped_refptr<TaskQueue> LoadingTaskQueue();
146 scoped_refptr<TaskQueue> TimerTaskQueue(); 148 scoped_refptr<TaskQueue> TimerTaskQueue();
147 149
148 // Returns a new loading task queue. This queue is intended for tasks related 150 // Returns a new loading task queue. This queue is intended for tasks related
149 // to resource dispatch, foreground HTML parsing, etc... 151 // to resource dispatch, foreground HTML parsing, etc...
150 scoped_refptr<TaskQueue> NewLoadingTaskQueue(TaskQueue::QueueType queue_type); 152 scoped_refptr<TaskQueue> NewLoadingTaskQueue(TaskQueue::QueueType queue_type);
151 153
(...skipping 391 matching lines...) Expand 10 before | Expand all | Expand 10 after
543 CompositorThreadOnly compositor_thread_only_; 545 CompositorThreadOnly compositor_thread_only_;
544 CompositorThreadOnly& GetCompositorThreadOnly() { 546 CompositorThreadOnly& GetCompositorThreadOnly() {
545 compositor_thread_only_.CheckOnValidThread(); 547 compositor_thread_only_.CheckOnValidThread();
546 return compositor_thread_only_; 548 return compositor_thread_only_;
547 } 549 }
548 550
549 PollableThreadSafeFlag policy_may_need_update_; 551 PollableThreadSafeFlag policy_may_need_update_;
550 552
551 base::WeakPtrFactory<RendererSchedulerImpl> weak_factory_; 553 base::WeakPtrFactory<RendererSchedulerImpl> weak_factory_;
552 554
555 base::TimeTicks uma_last_report_window_start_time_;
tdresser 2017/05/11 13:52:45 Let's make this be a bit more explicit. Maybe uma_
Liquan (Max) Gu 2017/05/11 15:14:13 Thanks! it's clearer.
556
553 DISALLOW_COPY_AND_ASSIGN(RendererSchedulerImpl); 557 DISALLOW_COPY_AND_ASSIGN(RendererSchedulerImpl);
554 }; 558 };
555 559
556 } // namespace scheduler 560 } // namespace scheduler
557 } // namespace blink 561 } // namespace blink
558 562
559 #endif // THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDU LER_IMPL_H_ 563 #endif // THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDU LER_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698