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

Unified Diff: base/task_scheduler/priority_queue_unittest.cc

Issue 2721553003: Remove SingleThreadTaskRunner Support from SchedulerWorkerPoolImpl (Closed)
Patch Set: Rebase to 08266b3 Created 3 years, 9 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: base/task_scheduler/priority_queue_unittest.cc
diff --git a/base/task_scheduler/priority_queue_unittest.cc b/base/task_scheduler/priority_queue_unittest.cc
index 3762856061f05fb6b66d757dd43cdfa0bdf16c94..221532b60306057cf6a2b64a2d547bfb8c142e55 100644
--- a/base/task_scheduler/priority_queue_unittest.cc
+++ b/base/task_scheduler/priority_queue_unittest.cc
@@ -141,19 +141,6 @@ TEST(TaskSchedulerPriorityQueueTest, IllegalTwoTransactionsSameThread) {
});
}
-// Check that there is no crash when Transactions are created on the same thread
-// for 2 PriorityQueues which have a predecessor relationship.
-TEST(TaskSchedulerPriorityQueueTest, LegalTwoTransactionsSameThread) {
- PriorityQueue pq_a;
- PriorityQueue pq_b(&pq_a);
-
- // This shouldn't crash.
- std::unique_ptr<PriorityQueue::Transaction> transaction_a =
- pq_a.BeginTransaction();
- std::unique_ptr<PriorityQueue::Transaction> transaction_b =
- pq_b.BeginTransaction();
-}
-
// Check that it is possible to begin multiple Transactions for the same
// PriorityQueue on different threads. The call to BeginTransaction() on the
// second thread should block until the Transaction has ended on the first

Powered by Google App Engine
This is Rietveld 408576698