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

Side by Side Diff: third_party/WebKit/Source/platform/scheduler/renderer/throttled_time_domain.cc

Issue 2572893002: [Reland] Dont post delayed DoWork for disabled queues. (Closed)
Patch Set: Fix perftest Created 4 years 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "platform/scheduler/renderer/throttled_time_domain.h" 5 #include "platform/scheduler/renderer/throttled_time_domain.h"
6 6
7 namespace blink { 7 namespace blink {
8 namespace scheduler { 8 namespace scheduler {
9 9
10 ThrottledTimeDomain::ThrottledTimeDomain(TimeDomain::Observer* observer, 10 ThrottledTimeDomain::ThrottledTimeDomain(TimeDomain::Observer* observer,
11 const char* tracing_category) 11 const char* tracing_category)
12 : RealTimeDomain(observer, tracing_category) {} 12 : RealTimeDomain(observer, tracing_category) {}
13 13
14 ThrottledTimeDomain::~ThrottledTimeDomain() {} 14 ThrottledTimeDomain::~ThrottledTimeDomain() {}
15 15
16 const char* ThrottledTimeDomain::GetName() const { 16 const char* ThrottledTimeDomain::GetName() const {
17 return "ThrottledTimeDomain"; 17 return "ThrottledTimeDomain";
18 } 18 }
19 19
20 void ThrottledTimeDomain::RequestWakeup(base::TimeTicks now, 20 void ThrottledTimeDomain::RequestWakeupAt(LazyNow* lazy_now,
21 base::TimeDelta delay) { 21 base::TimeTicks run_time) {
22 // We assume the owner (i.e. TaskQueueThrottler) will manage wakeups on our 22 // We assume the owner (i.e. TaskQueueThrottler) will manage wakeups on our
23 // behalf. 23 // behalf.
24 } 24 }
25 25
26 void ThrottledTimeDomain::CancelWakeupAt(base::TimeTicks run_time) {}
Sami 2016/12/14 16:29:38 // No-op because of the above.
alex clarke (OOO till 29th) 2016/12/14 16:50:36 Done.
27
26 base::Optional<base::TimeDelta> ThrottledTimeDomain::DelayTillNextTask( 28 base::Optional<base::TimeDelta> ThrottledTimeDomain::DelayTillNextTask(
27 LazyNow* lazy_now) { 29 LazyNow* lazy_now) {
28 base::TimeTicks next_run_time; 30 base::TimeTicks next_run_time;
29 if (!NextScheduledRunTime(&next_run_time)) 31 if (!NextScheduledRunTime(&next_run_time))
30 return base::Optional<base::TimeDelta>(); 32 return base::Optional<base::TimeDelta>();
31 33
32 base::TimeTicks now = lazy_now->Now(); 34 base::TimeTicks now = lazy_now->Now();
33 if (now >= next_run_time) 35 if (now >= next_run_time)
34 return base::TimeDelta(); // Makes DoWork post an immediate continuation. 36 return base::TimeDelta(); // Makes DoWork post an immediate continuation.
35 37
36 // Unlike RealTimeDomain::ContinuationNeeded we don't request a wake up here, 38 // Unlike RealTimeDomain::ContinuationNeeded we don't request a wake up here,
37 // we assume the owner (i.e. TaskQueueThrottler) will manage wakeups on our 39 // we assume the owner (i.e. TaskQueueThrottler) will manage wakeups on our
38 // behalf. 40 // behalf.
39 return base::Optional<base::TimeDelta>(); 41 return base::Optional<base::TimeDelta>();
40 } 42 }
41 43
42 } // namespace scheduler 44 } // namespace scheduler
43 } // namespace blink 45 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698