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

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

Issue 2393193003: Revert "[scheduler] Throttle background frames with 1% CPU limit" (Closed)
Patch Set: Created 4 years, 2 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/task_queue_throttler.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/renderer/task_queue_throttler.cc b/third_party/WebKit/Source/platform/scheduler/renderer/task_queue_throttler.cc
index 172decfff7b801c32d6cdcbcb8e0c6cf12c78ad5..d3ef352f0bb74f49478fa55f5da49f9161e9c526 100644
--- a/third_party/WebKit/Source/platform/scheduler/renderer/task_queue_throttler.cc
+++ b/third_party/WebKit/Source/platform/scheduler/renderer/task_queue_throttler.cc
@@ -88,16 +88,13 @@ void TaskQueueThrottler::TimeBudgetPool::SetTimeBudget(base::TimeTicks now,
void TaskQueueThrottler::TimeBudgetPool::AddQueue(base::TimeTicks now,
TaskQueue* queue) {
- std::pair<TaskQueueMap::iterator, bool> insert_result =
- task_queue_throttler_->queue_details_.insert(
- std::make_pair(queue, Metadata(0, queue->IsQueueEnabled())));
- Metadata& metadata = insert_result.first->second;
+ Metadata& metadata = task_queue_throttler_->queue_details_[queue];
DCHECK(!metadata.time_budget_pool);
metadata.time_budget_pool = this;
associated_task_queues_.insert(queue);
- if (!is_enabled_ || !metadata.IsThrottled())
+ if (!metadata.IsThrottled())
return;
queue->SetQueueEnabled(false);
@@ -117,7 +114,7 @@ void TaskQueueThrottler::TimeBudgetPool::RemoveQueue(base::TimeTicks now,
task_queue_throttler_->MaybeDeleteQueueMetadata(find_it);
associated_task_queues_.erase(queue);
- if (!is_enabled_ || !is_throttled)
+ if (is_throttled)
return;
task_queue_throttler_->MaybeSchedulePumpQueue(FROM_HERE, now, queue,

Powered by Google App Engine
This is Rietveld 408576698