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

Unified Diff: third_party/WebKit/Source/platform/scheduler/renderer/web_frame_scheduler_impl.cc

Issue 2741473002: [scheduler] Move TimeBudgetPool to a separate file. (Closed)
Patch Set: Addressed comments from alexclarke@ Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/scheduler/renderer/web_frame_scheduler_impl.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/renderer/web_frame_scheduler_impl.cc b/third_party/WebKit/Source/platform/scheduler/renderer/web_frame_scheduler_impl.cc
index b14a019da30d274670644eea6981c9285234b7c8..b6a266b135e7ec98221e6932e36902f0b438db04 100644
--- a/third_party/WebKit/Source/platform/scheduler/renderer/web_frame_scheduler_impl.cc
+++ b/third_party/WebKit/Source/platform/scheduler/renderer/web_frame_scheduler_impl.cc
@@ -6,13 +6,14 @@
#include "base/strings/stringprintf.h"
#include "base/trace_event/blame_context.h"
+#include "platform/RuntimeEnabledFeatures.h"
#include "platform/scheduler/base/real_time_domain.h"
#include "platform/scheduler/base/virtual_time_domain.h"
#include "platform/scheduler/child/web_task_runner_impl.h"
#include "platform/scheduler/renderer/auto_advancing_virtual_time_domain.h"
+#include "platform/scheduler/renderer/budget_pool.h"
#include "platform/scheduler/renderer/renderer_scheduler_impl.h"
#include "platform/scheduler/renderer/web_view_scheduler_impl.h"
-#include "platform/RuntimeEnabledFeatures.h"
#include "public/platform/BlameContext.h"
#include "public/platform/WebString.h"
@@ -63,7 +64,7 @@ WebFrameSchedulerImpl::~WebFrameSchedulerImpl() {
}
if (timer_task_queue_) {
- RemoveTimerQueueFromBackgroundTimeBudgetPool();
+ RemoveTimerQueueFromBackgroundCPUTimeBudgetPool();
timer_task_queue_->UnregisterTaskQueue();
timer_task_queue_->SetBlameContext(nullptr);
}
@@ -82,20 +83,20 @@ WebFrameSchedulerImpl::~WebFrameSchedulerImpl() {
}
void WebFrameSchedulerImpl::DetachFromWebViewScheduler() {
- RemoveTimerQueueFromBackgroundTimeBudgetPool();
+ RemoveTimerQueueFromBackgroundCPUTimeBudgetPool();
parent_web_view_scheduler_ = nullptr;
}
-void WebFrameSchedulerImpl::RemoveTimerQueueFromBackgroundTimeBudgetPool() {
+void WebFrameSchedulerImpl::RemoveTimerQueueFromBackgroundCPUTimeBudgetPool() {
if (!timer_task_queue_)
return;
if (!parent_web_view_scheduler_)
return;
- TaskQueueThrottler::TimeBudgetPool* time_budget_pool =
- parent_web_view_scheduler_->BackgroundTimeBudgetPool();
+ CPUTimeBudgetPool* time_budget_pool =
+ parent_web_view_scheduler_->BackgroundCPUTimeBudgetPool();
if (!time_budget_pool)
return;
@@ -145,8 +146,8 @@ RefPtr<blink::WebTaskRunner> WebFrameSchedulerImpl::timerTaskRunner() {
timer_queue_enabled_voter_ = timer_task_queue_->CreateQueueEnabledVoter();
timer_queue_enabled_voter_->SetQueueEnabled(!frame_suspended_);
- TaskQueueThrottler::TimeBudgetPool* time_budget_pool =
- parent_web_view_scheduler_->BackgroundTimeBudgetPool();
+ CPUTimeBudgetPool* time_budget_pool =
+ parent_web_view_scheduler_->BackgroundCPUTimeBudgetPool();
if (time_budget_pool) {
time_budget_pool->AddQueue(renderer_scheduler_->tick_clock()->NowTicks(),
timer_task_queue_.get());

Powered by Google App Engine
This is Rietveld 408576698