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

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

Issue 2780233002: Revert of [scheduler] Split suspendable tq from unthrottled tq. (Closed)
Patch Set: 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 6b5f081066b16edfbb556cb37ec813b3bdd156d5..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
@@ -74,11 +74,6 @@
unthrottled_task_queue_->SetBlameContext(nullptr);
}
- if (suspendable_task_queue_) {
- suspendable_task_queue_->UnregisterTaskQueue();
- suspendable_task_queue_->SetBlameContext(nullptr);
- }
-
if (parent_web_view_scheduler_) {
parent_web_view_scheduler_->Unregister(this);
@@ -167,23 +162,6 @@
return timer_web_task_runner_;
}
-RefPtr<blink::WebTaskRunner> WebFrameSchedulerImpl::suspendableTaskRunner() {
- DCHECK(parent_web_view_scheduler_);
- if (!suspendable_web_task_runner_) {
- // TODO(altimin): Split FRAME_UNTHROTTLED into FRAME_UNTHROTTLED and
- // FRAME_UNSUSPENDED.
- suspendable_task_queue_ = renderer_scheduler_->NewTimerTaskRunner(
- TaskQueue::QueueType::FRAME_UNTHROTTLED);
- suspendable_task_queue_->SetBlameContext(blame_context_);
- suspendable_web_task_runner_ =
- WebTaskRunnerImpl::create(suspendable_task_queue_);
- suspendable_queue_enabled_voter_ =
- suspendable_task_queue_->CreateQueueEnabledVoter();
- suspendable_queue_enabled_voter_->SetQueueEnabled(!frame_suspended_);
- }
- return suspendable_web_task_runner_;
-}
-
RefPtr<blink::WebTaskRunner> WebFrameSchedulerImpl::unthrottledTaskRunner() {
DCHECK(parent_web_view_scheduler_);
if (!unthrottled_web_task_runner_) {
@@ -246,10 +224,6 @@
state->SetString("unthrottled_task_queue",
PointerToId(unthrottled_task_queue_.get()));
}
- if (suspendable_task_queue_) {
- state->SetString("suspendable_task_queue",
- PointerToId(suspendable_task_queue_.get()));
- }
if (blame_context_) {
state->BeginDictionary("blame_context");
state->SetString(
@@ -278,8 +252,6 @@
loading_queue_enabled_voter_->SetQueueEnabled(!frame_suspended);
if (timer_queue_enabled_voter_)
timer_queue_enabled_voter_->SetQueueEnabled(!frame_suspended);
- if (suspendable_queue_enabled_voter_)
- suspendable_queue_enabled_voter_->SetQueueEnabled(!frame_suspended);
}
void WebFrameSchedulerImpl::onFirstMeaningfulPaint() {

Powered by Google App Engine
This is Rietveld 408576698