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

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

Issue 2419793002: [Reland] Optimize blink scheduler with an intrusive heap (Closed)
Patch Set: Fixed perf test and increaced test coverage 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();
161 } 165 }
162 166
163 TEST_F(TimeDomainTest, UnregisterQueue) { 167 TEST_F(TimeDomainTest, UnregisterQueue) {
164 scoped_refptr<internal::TaskQueueImpl> task_queue2_ = 168 scoped_refptr<internal::TaskQueueImpl> task_queue2_ =
165 make_scoped_refptr(new internal::TaskQueueImpl( 169 make_scoped_refptr(new internal::TaskQueueImpl(
166 nullptr, time_domain_.get(), TaskQueue::Spec("test_queue2"), 170 nullptr, time_domain_.get(), TaskQueue::Spec("test_queue2"),
167 "test.category", "test.category")); 171 "test.category", "test.category"));
168 172
169 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, _)).Times(1); 173 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, _)).Times(1);
170 base::TimeTicks now = time_domain_->Now(); 174 base::TimeTicks now = time_domain_->Now();
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
236 TEST_F(TimeDomainWithObserverTest, OnTimeDomainHasDelayedWork) { 240 TEST_F(TimeDomainWithObserverTest, OnTimeDomainHasDelayedWork) {
237 EXPECT_CALL(*observer_, OnTimeDomainHasDelayedWork(task_queue_.get())); 241 EXPECT_CALL(*observer_, OnTimeDomainHasDelayedWork(task_queue_.get()));
238 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, _)); 242 EXPECT_CALL(*time_domain_.get(), RequestWakeup(_, _));
239 base::TimeTicks now = time_domain_->Now(); 243 base::TimeTicks now = time_domain_->Now();
240 time_domain_->ScheduleDelayedWork( 244 time_domain_->ScheduleDelayedWork(
241 task_queue_.get(), now + base::TimeDelta::FromMilliseconds(10), now); 245 task_queue_.get(), now + base::TimeDelta::FromMilliseconds(10), now);
242 } 246 }
243 247
244 } // namespace scheduler 248 } // namespace scheduler
245 } // namespace blink 249 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698