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

Unified Diff: third_party/WebKit/Source/platform/scheduler/base/work_queue_sets_unittest.cc

Issue 2786083005: scheduler: Maintain a constant enqueue order for every task (Closed)
Patch Set: WASM workaround no longer needed Created 3 years, 8 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/base/work_queue_sets_unittest.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/base/work_queue_sets_unittest.cc b/third_party/WebKit/Source/platform/scheduler/base/work_queue_sets_unittest.cc
index de0b9c4124f5b436db80a12e9f211495b0c3abb4..057d1ac77524f43cd9e1ac786c2df9fbea47e9f9 100644
--- a/third_party/WebKit/Source/platform/scheduler/base/work_queue_sets_unittest.cc
+++ b/third_party/WebKit/Source/platform/scheduler/base/work_queue_sets_unittest.cc
@@ -42,10 +42,9 @@ class WorkQueueSetsTest : public testing::Test {
return queue;
}
- TaskQueueImpl::Task FakeTaskWithEnqueueOrder(int enqueue_order) {
- TaskQueueImpl::Task fake_task(FROM_HERE, base::Closure(), base::TimeTicks(),
- 0, true);
- fake_task.set_enqueue_order(enqueue_order);
+ TaskQueueImpl::Task FakeTaskWithEnqueueOrder(int sequence_number) {
+ TaskQueueImpl::Task fake_task(FROM_HERE, base::Closure(),
+ {base::TimeTicks(), sequence_number}, true);
return fake_task;
}
@@ -109,7 +108,7 @@ TEST_F(WorkQueueSetsTest, GetOldestQueueAndEnqueueOrderInSet) {
EXPECT_TRUE(work_queue_sets_->GetOldestQueueAndEnqueueOrderInSet(
set, &selected_work_queue, &enqueue_order));
EXPECT_EQ(work_queue, selected_work_queue);
- EXPECT_EQ(10u, enqueue_order);
+ EXPECT_EQ(10u, enqueue_order.sequence_num);
}
TEST_F(WorkQueueSetsTest, GetOldestQueueInSet_MultipleAgesInSet) {

Powered by Google App Engine
This is Rietveld 408576698