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

Unified Diff: third_party/WebKit/Source/platform/scheduler/renderer/web_view_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
« no previous file with comments | « third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.cc b/third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.cc
index d14a7f742d79c02cd9c19e38ec5218e3022d287d..b6a1a3ebf04904780455541b0ee980fd255effdd 100644
--- a/third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.cc
+++ b/third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.cc
@@ -11,6 +11,7 @@
#include "platform/scheduler/base/virtual_time_domain.h"
#include "platform/scheduler/child/scheduler_tqm_delegate.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_frame_scheduler_impl.h"
@@ -142,7 +143,7 @@ void WebViewSchedulerImpl::setPageVisible(bool page_visible) {
std::unique_ptr<WebFrameSchedulerImpl>
WebViewSchedulerImpl::createWebFrameSchedulerImpl(
base::trace_event::BlameContext* blame_context) {
- MaybeInitializeBackgroundTimeBudgetPool();
+ MaybeInitializeBackgroundCPUTimeBudgetPool();
std::unique_ptr<WebFrameSchedulerImpl> frame_scheduler(
new WebFrameSchedulerImpl(renderer_scheduler_, this, blame_context));
frame_scheduler->setPageThrottled(should_throttle_frames_);
@@ -298,13 +299,12 @@ void WebViewSchedulerImpl::AsValueInto(
state->EndDictionary();
}
-TaskQueueThrottler::TimeBudgetPool*
-WebViewSchedulerImpl::BackgroundTimeBudgetPool() {
- MaybeInitializeBackgroundTimeBudgetPool();
+CPUTimeBudgetPool* WebViewSchedulerImpl::BackgroundCPUTimeBudgetPool() {
+ MaybeInitializeBackgroundCPUTimeBudgetPool();
return background_time_budget_pool_;
}
-void WebViewSchedulerImpl::MaybeInitializeBackgroundTimeBudgetPool() {
+void WebViewSchedulerImpl::MaybeInitializeBackgroundCPUTimeBudgetPool() {
if (background_time_budget_pool_)
return;
@@ -312,7 +312,7 @@ void WebViewSchedulerImpl::MaybeInitializeBackgroundTimeBudgetPool() {
return;
background_time_budget_pool_ =
- renderer_scheduler_->task_queue_throttler()->CreateTimeBudgetPool(
+ renderer_scheduler_->task_queue_throttler()->CreateCPUTimeBudgetPool(
"background", GetMaxBudgetLevel(settings_),
GetMaxThrottlingDelay(settings_));
« no previous file with comments | « third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698