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

Side by Side Diff: third_party/WebKit/Source/platform/scheduler/base/time_domain_unittest.cc

Issue 2421283002: Revert of Optimize blink scheduler with an intrusive heap (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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/base/time_domain.h" 5 #include "platform/scheduler/base/time_domain.h"
6 6
7 #include "base/macros.h" 7 #include "base/macros.h"
8 #include "base/memory/ptr_util.h" 8 #include "base/memory/ptr_util.h"
9 #include "base/test/simple_test_tick_clock.h" 9 #include "base/test/simple_test_tick_clock.h"
10 #include "cc/test/ordered_simple_task_runner.h" 10 #include "cc/test/ordered_simple_task_runner.h"
(...skipping 140 matching lines...) Expand 10 before | Expand all | Expand 10 after
151 151
152 // RequestWakeup should not be called when scheduling later tasks. 152 // RequestWakeup should not be called when scheduling later tasks.
153 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, _)).Times(0); 153 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, _)).Times(0);
154 time_domain_->ScheduleDelayedWork(task_queue2.get(), now + delay2, now); 154 time_domain_->ScheduleDelayedWork(task_queue2.get(), now + delay2, now);
155 time_domain_->ScheduleDelayedWork(task_queue3.get(), now + delay3, now); 155 time_domain_->ScheduleDelayedWork(task_queue3.get(), now + delay3, now);
156 156
157 // RequestWakeup should be called when scheduling earlier tasks. 157 // RequestWakeup should be called when scheduling earlier tasks.
158 Mock::VerifyAndClearExpectations(time_domain_.get()); 158 Mock::VerifyAndClearExpectations(time_domain_.get());
159 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, delay4)); 159 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, delay4));
160 time_domain_->ScheduleDelayedWork(task_queue4.get(), now + delay4, now); 160 time_domain_->ScheduleDelayedWork(task_queue4.get(), now + delay4, now);
161
162 task_queue2->UnregisterTaskQueue();
163 task_queue3->UnregisterTaskQueue();
164 task_queue4->UnregisterTaskQueue();
165 } 161 }
166 162
167 TEST_F(TimeDomainTest, UnregisterQueue) { 163 TEST_F(TimeDomainTest, UnregisterQueue) {
168 scoped_refptr<internal::TaskQueueImpl> task_queue2_ = 164 scoped_refptr<internal::TaskQueueImpl> task_queue2_ =
169 make_scoped_refptr(new internal::TaskQueueImpl( 165 make_scoped_refptr(new internal::TaskQueueImpl(
170 nullptr, time_domain_.get(), TaskQueue::Spec("test_queue2"), 166 nullptr, time_domain_.get(), TaskQueue::Spec("test_queue2"),
171 "test.category", "test.category")); 167 "test.category", "test.category"));
172 168
173 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, _)).Times(1); 169 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, _)).Times(1);
174 base::TimeTicks now = time_domain_->Now(); 170 base::TimeTicks now = time_domain_->Now();
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
240 TEST_F(TimeDomainWithObserverTest, OnTimeDomainHasDelayedWork) { 236 TEST_F(TimeDomainWithObserverTest, OnTimeDomainHasDelayedWork) {
241 EXPECT_CALL(*observer_, OnTimeDomainHasDelayedWork(task_queue_.get())); 237 EXPECT_CALL(*observer_, OnTimeDomainHasDelayedWork(task_queue_.get()));
242 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, _)); 238 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, _));
243 base::TimeTicks now = time_domain_->Now(); 239 base::TimeTicks now = time_domain_->Now();
244 time_domain_->ScheduleDelayedWork( 240 time_domain_->ScheduleDelayedWork(
245 task_queue_.get(), now + base::TimeDelta::FromMilliseconds(10), now); 241 task_queue_.get(), now + base::TimeDelta::FromMilliseconds(10), now);
246 } 242 }
247 243
248 } // namespace scheduler 244 } // namespace scheduler
249 } // namespace blink 245 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698