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

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

Issue 2788013003: Expected Queueing Time Metric ignores tasks with nested message loops. (Closed)
Patch Set: Address nits Created 3 years, 8 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 127 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 // SchedulerHelper::Observer implementation: 138 // SchedulerHelper::Observer implementation:
139 void OnUnregisterTaskQueue(const scoped_refptr<TaskQueue>& queue) override; 139 void OnUnregisterTaskQueue(const scoped_refptr<TaskQueue>& queue) override;
140 void OnTriedToExecuteBlockedTask(const TaskQueue& queue, 140 void OnTriedToExecuteBlockedTask(const TaskQueue& queue,
141 const base::PendingTask& task) override; 141 const base::PendingTask& task) override;
142 142
143 // TaskTimeObserver implementation: 143 // TaskTimeObserver implementation:
144 void willProcessTask(TaskQueue* task_queue, double start_time) override; 144 void willProcessTask(TaskQueue* task_queue, double start_time) override;
145 void didProcessTask(TaskQueue* task_queue, 145 void didProcessTask(TaskQueue* task_queue,
146 double start_time, 146 double start_time,
147 double end_time) override; 147 double end_time) override;
148 void onBeginNestedMessageLoop() override;
148 149
149 // QueueingTimeEstimator::Client implementation: 150 // QueueingTimeEstimator::Client implementation:
150 void OnQueueingTimeForWindowEstimated(base::TimeDelta queueing_time) override; 151 void OnQueueingTimeForWindowEstimated(base::TimeDelta queueing_time) override;
151 152
152 // Returns a task runner where tasks run at the highest possible priority. 153 // Returns a task runner where tasks run at the highest possible priority.
153 scoped_refptr<TaskQueue> ControlTaskRunner(); 154 scoped_refptr<TaskQueue> ControlTaskRunner();
154 155
155 void RegisterTimeDomain(TimeDomain* time_domain); 156 void RegisterTimeDomain(TimeDomain* time_domain);
156 void UnregisterTimeDomain(TimeDomain* time_domain); 157 void UnregisterTimeDomain(TimeDomain* time_domain);
157 158
(...skipping 372 matching lines...) Expand 10 before | Expand all | Expand 10 after
530 531
531 base::WeakPtrFactory<RendererSchedulerImpl> weak_factory_; 532 base::WeakPtrFactory<RendererSchedulerImpl> weak_factory_;
532 533
533 DISALLOW_COPY_AND_ASSIGN(RendererSchedulerImpl); 534 DISALLOW_COPY_AND_ASSIGN(RendererSchedulerImpl);
534 }; 535 };
535 536
536 } // namespace scheduler 537 } // namespace scheduler
537 } // namespace blink 538 } // namespace blink
538 539
539 #endif // THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDU LER_IMPL_H_ 540 #endif // THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_RENDERER_RENDERER_SCHEDU LER_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698