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

Unified Diff: third_party/WebKit/Source/platform/scheduler/child/compositor_worker_scheduler.cc

Issue 2276353002: Remove after wakeup logic and replace PumpTask with Fences (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Slight simplification Created 4 years, 3 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/child/compositor_worker_scheduler.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/child/compositor_worker_scheduler.cc b/third_party/WebKit/Source/platform/scheduler/child/compositor_worker_scheduler.cc
index a2608800f4ec91158770cf49945ef3b6f8faef93..0c654ee77e942a618665fac72a4d39af8ac30af4 100644
--- a/third_party/WebKit/Source/platform/scheduler/child/compositor_worker_scheduler.cc
+++ b/third_party/WebKit/Source/platform/scheduler/child/compositor_worker_scheduler.cc
@@ -57,6 +57,15 @@ class CompositorWorkerTaskRunnerWrapper : public TaskQueue {
void SetQueueEnabled(bool enabled) override { NOTREACHED(); }
+ void InsertFence() override { NOTREACHED(); }
+
+ void RemoveFence() override { NOTREACHED(); }
+
+ bool BlockedByFence() const override {
+ NOTREACHED();
+ return false;
+ }
+
bool IsQueueEnabled() const override {
NOTREACHED();
return true;
@@ -72,11 +81,6 @@ class CompositorWorkerTaskRunnerWrapper : public TaskQueue {
return false;
};
- bool NeedsPumping() const override {
- NOTREACHED();
- return false;
- };
-
const char* GetName() const override {
NOTREACHED();
return nullptr;
@@ -89,15 +93,6 @@ class CompositorWorkerTaskRunnerWrapper : public TaskQueue {
return QueuePriority::NORMAL_PRIORITY;
};
- void SetPumpPolicy(PumpPolicy pump_policy) override { NOTREACHED(); }
-
- PumpPolicy GetPumpPolicy() const override {
- NOTREACHED();
- return PumpPolicy::AUTO;
- };
-
- void PumpQueue(LazyNow*, bool may_post_dowork) override { NOTREACHED(); }
-
void AddTaskObserver(
base::MessageLoop::TaskObserver* task_observer) override {
NOTREACHED();
@@ -146,8 +141,7 @@ CompositorWorkerScheduler::IdleTaskRunner() {
// which runs them after the current frame has been drawn before the next
// vsync. https://crbug.com/609532
return make_scoped_refptr(new SingleThreadIdleTaskRunner(
- thread_->task_runner(), thread_->task_runner(), this,
- "compositor.scheduler"));
+ thread_->task_runner(), this, "compositor.scheduler"));
}
bool CompositorWorkerScheduler::CanExceedIdleDeadlineIfRequired() const {

Powered by Google App Engine
This is Rietveld 408576698