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

Unified Diff: base/task_scheduler/scheduler_thread_pool.cc

Issue 1901223003: base: Remove a copy of and copy ctor from SequenceAndSortKey. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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: base/task_scheduler/scheduler_thread_pool.cc
diff --git a/base/task_scheduler/scheduler_thread_pool.cc b/base/task_scheduler/scheduler_thread_pool.cc
index b00990bb82945e8026f5cb5ad263462e39277863..f7e24b11aa75dba4e6624123f1c50a3d33ce4b45 100644
--- a/base/task_scheduler/scheduler_thread_pool.cc
+++ b/base/task_scheduler/scheduler_thread_pool.cc
@@ -238,7 +238,7 @@ SchedulerThreadPool::SchedulerWorkerThreadDelegateImpl::GetWork(
SchedulerWorkerThread* worker_thread) {
std::unique_ptr<PriorityQueue::Transaction> transaction(
outer_->shared_priority_queue_.BeginTransaction());
- const auto sequence_and_sort_key = transaction->Peek();
+ const auto& sequence_and_sort_key = transaction->Peek();
if (sequence_and_sort_key.is_null()) {
// |transaction| is kept alive while |worker_thread| is added to
« base/task_scheduler/priority_queue.h ('K') | « base/task_scheduler/priority_queue.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698