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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: components/scheduler/child/task_queue_sets_unittest.cc

Issue 1259583006: Reland: Explicitly track the scheduler task enqueueing order in a new field (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix comment Created 5 years, 4 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
« components/scheduler/child/task_queue_manager.h ('K') | « components/scheduler/child/task_queue_selector_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/scheduler/child/task_queue_sets_unittest.cc
diff --git a/components/scheduler/child/task_queue_sets_unittest.cc b/components/scheduler/child/task_queue_sets_unittest.cc
index cf7959891a82baa2715671b9f084f42fc1ae7bb5..c510c1dee387ba6ff6bb14a52ddcd69953edafcb 100644
--- a/components/scheduler/child/task_queue_sets_unittest.cc
+++ b/components/scheduler/child/task_queue_sets_unittest.cc
@@ -27,9 +27,9 @@ class TaskQueueSetsTest : public testing::Test {
return queue.get();
}
- base::PendingTask FakeTaskWithSequenceNum(int sequence_num) {
- base::PendingTask fake_task(FROM_HERE, base::Closure());
- fake_task.sequence_num = sequence_num;
+ SchedulerTask FakeTaskWithAge(int age) {
+ SchedulerTask fake_task(FROM_HERE, base::Closure(), true);
+ fake_task.age = age;
return fake_task;
}
@@ -62,7 +62,7 @@ TEST_F(TaskQueueSetsTest, OnPushQueue) {
internal::TaskQueueImpl* selected_queue;
EXPECT_FALSE(task_queue_sets_->GetOldestQueueInSet(set, &selected_queue));
- queue->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(10));
+ queue->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(10));
task_queue_sets_->OnPushQueue(queue);
EXPECT_TRUE(task_queue_sets_->GetOldestQueueInSet(set, &selected_queue));
@@ -71,7 +71,7 @@ TEST_F(TaskQueueSetsTest, OnPushQueue) {
TEST_F(TaskQueueSetsTest, GetOldestQueueInSet_SingleTaskInSet) {
internal::TaskQueueImpl* queue = NewTaskQueue();
- queue->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(10));
+ queue->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(10));
size_t set = 1;
task_queue_sets_->AssignQueueToSet(queue, set);
@@ -84,9 +84,9 @@ TEST_F(TaskQueueSetsTest, GetOldestQueueInSet_MultipleAgesInSet) {
internal::TaskQueueImpl* queue1 = NewTaskQueue();
internal::TaskQueueImpl* queue2 = NewTaskQueue();
internal::TaskQueueImpl* queue3 = NewTaskQueue();
- queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(6));
- queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(5));
- queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(4));
+ queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(6));
+ queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(5));
+ queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(4));
size_t set = 2;
task_queue_sets_->AssignQueueToSet(queue1, set);
task_queue_sets_->AssignQueueToSet(queue2, set);
@@ -101,10 +101,10 @@ TEST_F(TaskQueueSetsTest, OnPopQueue) {
internal::TaskQueueImpl* queue1 = NewTaskQueue();
internal::TaskQueueImpl* queue2 = NewTaskQueue();
internal::TaskQueueImpl* queue3 = NewTaskQueue();
- queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(6));
- queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(3));
- queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(1));
- queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(4));
+ queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(6));
+ queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(3));
+ queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(1));
+ queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(4));
size_t set = 3;
task_queue_sets_->AssignQueueToSet(queue1, set);
task_queue_sets_->AssignQueueToSet(queue2, set);
@@ -125,9 +125,9 @@ TEST_F(TaskQueueSetsTest, OnPopQueue_QueueBecomesEmpty) {
internal::TaskQueueImpl* queue1 = NewTaskQueue();
internal::TaskQueueImpl* queue2 = NewTaskQueue();
internal::TaskQueueImpl* queue3 = NewTaskQueue();
- queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(6));
- queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(5));
- queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(4));
+ queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(6));
+ queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(5));
+ queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(4));
size_t set = 4;
task_queue_sets_->AssignQueueToSet(queue1, set);
task_queue_sets_->AssignQueueToSet(queue2, set);
@@ -149,9 +149,9 @@ TEST_F(TaskQueueSetsTest,
internal::TaskQueueImpl* queue1 = NewTaskQueue();
internal::TaskQueueImpl* queue2 = NewTaskQueue();
internal::TaskQueueImpl* queue3 = NewTaskQueue();
- queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(0x7ffffff1));
- queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(0x7ffffff0));
- queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(-0x7ffffff1));
+ queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(0x7ffffff1));
+ queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(0x7ffffff0));
+ queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(-0x7ffffff1));
size_t set = 0;
task_queue_sets_->AssignQueueToSet(queue1, set);
task_queue_sets_->AssignQueueToSet(queue2, set);
@@ -166,9 +166,9 @@ TEST_F(TaskQueueSetsTest, GetOldestQueueInSet_MultipleAgesInSet_RemoveQueue) {
internal::TaskQueueImpl* queue1 = NewTaskQueue();
internal::TaskQueueImpl* queue2 = NewTaskQueue();
internal::TaskQueueImpl* queue3 = NewTaskQueue();
- queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(6));
- queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(5));
- queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(4));
+ queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(6));
+ queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(5));
+ queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(4));
size_t set = 1;
task_queue_sets_->AssignQueueToSet(queue1, set);
task_queue_sets_->AssignQueueToSet(queue2, set);
@@ -185,10 +185,10 @@ TEST_F(TaskQueueSetsTest, AssignQueueToSet_Complex) {
internal::TaskQueueImpl* queue2 = NewTaskQueue();
internal::TaskQueueImpl* queue3 = NewTaskQueue();
internal::TaskQueueImpl* queue4 = NewTaskQueue();
- queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(6));
- queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(5));
- queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(4));
- queue4->PushTaskOntoWorkQueueForTest(FakeTaskWithSequenceNum(3));
+ queue1->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(6));
+ queue2->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(5));
+ queue3->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(4));
+ queue4->PushTaskOntoWorkQueueForTest(FakeTaskWithAge(3));
size_t set1 = 1;
size_t set2 = 2;
task_queue_sets_->AssignQueueToSet(queue1, set1);
« components/scheduler/child/task_queue_manager.h ('K') | « components/scheduler/child/task_queue_selector_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698