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

Unified Diff: third_party/WebKit/Source/platform/scheduler/renderer/budget_pool.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/budget_pool.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/renderer/budget_pool.cc b/third_party/WebKit/Source/platform/scheduler/renderer/budget_pool.cc
new file mode 100644
index 0000000000000000000000000000000000000000..3219540bcf3ef3b850d4032286fca527694fe0b4
--- /dev/null
+++ b/third_party/WebKit/Source/platform/scheduler/renderer/budget_pool.cc
@@ -0,0 +1,239 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "platform/scheduler/renderer/budget_pool.h"
+
+#include <cstdint>
+
+#include "base/format_macros.h"
+#include "base/logging.h"
+#include "base/memory/ptr_util.h"
+#include "base/optional.h"
+#include "base/strings/stringprintf.h"
+#include "platform/WebFrameScheduler.h"
+#include "platform/scheduler/base/real_time_domain.h"
+#include "platform/scheduler/child/scheduler_tqm_delegate.h"
+#include "platform/scheduler/renderer/renderer_scheduler_impl.h"
+#include "platform/scheduler/renderer/task_queue_throttler.h"
+#include "platform/scheduler/renderer/throttled_time_domain.h"
+#include "platform/scheduler/renderer/web_frame_scheduler_impl.h"
+
+namespace blink {
+namespace scheduler {
+
+namespace {
+
+std::string PointerToId(void* pointer) {
+ return base::StringPrintf(
+ "0x%" PRIx64,
+ static_cast<uint64_t>(reinterpret_cast<uintptr_t>(pointer)));
+}
+
+} // namespace
+
+BudgetPool::~BudgetPool() {}
+
+CPUTimeBudgetPool::CPUTimeBudgetPool(
+ const char* name,
+ TaskQueueThrottler* task_queue_throttler,
+ base::TimeTicks now,
+ base::Optional<base::TimeDelta> max_budget_level,
+ base::Optional<base::TimeDelta> max_throttling_duration)
+ : name_(name),
+ task_queue_throttler_(task_queue_throttler),
+ max_budget_level_(max_budget_level),
+ max_throttling_duration_(max_throttling_duration),
+ last_checkpoint_(now),
+ cpu_percentage_(1),
+ is_enabled_(true) {}
+
+CPUTimeBudgetPool::~CPUTimeBudgetPool() {}
+
+void CPUTimeBudgetPool::SetTimeBudgetRecoveryRate(base::TimeTicks now,
+ double cpu_percentage) {
+ Advance(now);
+ cpu_percentage_ = cpu_percentage;
+ EnforceBudgetLevelRestrictions();
+}
+
+void CPUTimeBudgetPool::AddQueue(base::TimeTicks now, TaskQueue* queue) {
+ std::pair<TaskQueueThrottler::TaskQueueMap::iterator, bool> insert_result =
+ task_queue_throttler_->queue_details_.insert(
+ std::make_pair(queue, TaskQueueThrottler::Metadata()));
+ TaskQueueThrottler::Metadata& metadata = insert_result.first->second;
+ DCHECK(!metadata.time_budget_pool);
+ metadata.time_budget_pool = this;
+
+ associated_task_queues_.insert(queue);
+
+ if (!is_enabled_ || !task_queue_throttler_->IsThrottled(queue))
+ return;
+
+ queue->InsertFence(TaskQueue::InsertFencePosition::BEGINNING_OF_TIME);
+
+ task_queue_throttler_->MaybeSchedulePumpQueue(FROM_HERE, now, queue,
+ GetNextAllowedRunTime());
+}
+
+void CPUTimeBudgetPool::RemoveQueue(base::TimeTicks now, TaskQueue* queue) {
+ auto find_it = task_queue_throttler_->queue_details_.find(queue);
+ DCHECK(find_it != task_queue_throttler_->queue_details_.end() &&
+ find_it->second.time_budget_pool == this);
+ find_it->second.time_budget_pool = nullptr;
+ bool is_throttled = task_queue_throttler_->IsThrottled(queue);
+
+ task_queue_throttler_->MaybeDeleteQueueMetadata(find_it);
+ associated_task_queues_.erase(queue);
+
+ if (!is_enabled_ || !is_throttled)
+ return;
+
+ task_queue_throttler_->MaybeSchedulePumpQueue(FROM_HERE, now, queue,
+ base::nullopt);
+}
+
+void CPUTimeBudgetPool::EnableThrottling(LazyNow* lazy_now) {
+ if (is_enabled_)
+ return;
+ is_enabled_ = true;
+
+ TRACE_EVENT0(task_queue_throttler_->tracing_category_,
+ "CPUTimeBudgetPool_EnableThrottling");
+
+ BlockThrottledQueues(lazy_now->Now());
+}
+
+void CPUTimeBudgetPool::DisableThrottling(LazyNow* lazy_now) {
+ if (!is_enabled_)
+ return;
+ is_enabled_ = false;
+
+ TRACE_EVENT0(task_queue_throttler_->tracing_category_,
+ "CPUTimeBudgetPool_DisableThrottling");
+
+ for (TaskQueue* queue : associated_task_queues_) {
+ if (!task_queue_throttler_->IsThrottled(queue))
+ continue;
+
+ task_queue_throttler_->MaybeSchedulePumpQueue(FROM_HERE, lazy_now->Now(),
+ queue, base::nullopt);
+ }
+
+ // TODO(altimin): We need to disable TimeBudgetQueues here or they will
+ // regenerate extra time budget when they are disabled.
+}
+
+bool CPUTimeBudgetPool::IsThrottlingEnabled() const {
+ return is_enabled_;
+}
+
+void CPUTimeBudgetPool::GrantAdditionalBudget(base::TimeTicks now,
+ base::TimeDelta budget_level) {
+ Advance(now);
+ current_budget_level_ += budget_level;
+ EnforceBudgetLevelRestrictions();
+}
+
+void CPUTimeBudgetPool::SetReportingCallback(
+ base::Callback<void(base::TimeDelta)> reporting_callback) {
+ reporting_callback_ = reporting_callback;
+}
+
+void CPUTimeBudgetPool::Close() {
+ DCHECK_EQ(0u, associated_task_queues_.size());
+
+ task_queue_throttler_->time_budget_pools_.erase(this);
+}
+
+bool CPUTimeBudgetPool::HasEnoughBudgetToRun(base::TimeTicks now) {
+ Advance(now);
+ return !is_enabled_ || current_budget_level_.InMicroseconds() >= 0;
+}
+
+base::TimeTicks CPUTimeBudgetPool::GetNextAllowedRunTime() {
+ if (!is_enabled_ || current_budget_level_.InMicroseconds() >= 0) {
+ return last_checkpoint_;
+ } else {
+ // Subtract because current_budget is negative.
+ return last_checkpoint_ - current_budget_level_ / cpu_percentage_;
+ }
+}
+
+void CPUTimeBudgetPool::RecordTaskRunTime(base::TimeTicks start_time,
+ base::TimeTicks end_time) {
+ DCHECK_LE(start_time, end_time);
+ Advance(end_time);
+ if (is_enabled_) {
+ base::TimeDelta old_budget_level = current_budget_level_;
+ current_budget_level_ -= (end_time - start_time);
+ EnforceBudgetLevelRestrictions();
+
+ if (!reporting_callback_.is_null() && old_budget_level.InSecondsF() > 0 &&
+ current_budget_level_.InSecondsF() < 0) {
+ reporting_callback_.Run(-current_budget_level_ / cpu_percentage_);
+ }
+ }
+}
+
+const char* CPUTimeBudgetPool::Name() const {
+ return name_;
+}
+
+void CPUTimeBudgetPool::AsValueInto(base::trace_event::TracedValue* state,
+ base::TimeTicks now) const {
+ state->BeginDictionary(name_);
+
+ state->SetString("name", name_);
+ state->SetDouble("time_budget", cpu_percentage_);
+ state->SetDouble("time_budget_level_in_seconds",
+ current_budget_level_.InSecondsF());
+ state->SetDouble("last_checkpoint_seconds_ago",
+ (now - last_checkpoint_).InSecondsF());
+ state->SetBoolean("is_enabled", is_enabled_);
+
+ state->BeginArray("task_queues");
+ for (TaskQueue* queue : associated_task_queues_) {
+ state->AppendString(PointerToId(queue));
+ }
+ state->EndArray();
+
+ state->EndDictionary();
+}
+
+void CPUTimeBudgetPool::Advance(base::TimeTicks now) {
+ if (now > last_checkpoint_) {
+ if (is_enabled_) {
+ current_budget_level_ += cpu_percentage_ * (now - last_checkpoint_);
+ EnforceBudgetLevelRestrictions();
+ }
+ last_checkpoint_ = now;
+ }
+}
+
+void CPUTimeBudgetPool::BlockThrottledQueues(base::TimeTicks now) {
+ for (TaskQueue* queue : associated_task_queues_) {
+ if (!task_queue_throttler_->IsThrottled(queue))
+ continue;
+
+ queue->InsertFence(TaskQueue::InsertFencePosition::BEGINNING_OF_TIME);
+ task_queue_throttler_->MaybeSchedulePumpQueue(FROM_HERE, now, queue,
+ base::nullopt);
+ }
+}
+
+void CPUTimeBudgetPool::EnforceBudgetLevelRestrictions() {
+ if (max_budget_level_) {
+ current_budget_level_ =
+ std::min(current_budget_level_, max_budget_level_.value());
+ }
+ if (max_throttling_duration_) {
+ // Current budget level may be negative.
+ current_budget_level_ =
+ std::max(current_budget_level_,
+ -max_throttling_duration_.value() * cpu_percentage_);
+ }
+}
+
+} // namespace scheduler
+} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698