TaskScheduler [6] SchedulerWorkerThread

base/task_scheduler/utils.cc

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "base/task_scheduler/utils.h"
+
+#include "base/bind.h"
+#include "base/task_scheduler/priority_queue.h"
+#include "base/task_scheduler/task_tracker.h"
+
+namespace base {
+namespace internal {
+
+namespace {
+
+void PostTaskCallback(scoped_refptr<Sequence> sequence,
+                      PriorityQueue* priority_queue,
+                      scoped_ptr<Task> task) {
+  if (sequence->PushTask(std::move(task))) {
+    // |sequence| was empty before |task| was inserted into it. That means that
+    // |sequence| must be inserted in |priority_queue|.
+    SequenceSortKey sequence_sort_key = sequence->GetSortKey();

[gab, 2016/03/21 19:11:53]

const

[fdoray, 2016/03/24 19:21:09]

Done.

+    priority_queue->BeginTransaction()->Push(make_scoped_ptr(
+        new PriorityQueue::SequenceAndSortKey(sequence, sequence_sort_key)));

[gab, 2016/03/21 19:11:53]

std::move(sequence)

Not sure whether compiler can figure out that this is movable as it's not
referenced after but moving it makes the intent explicit.

[fdoray, 2016/03/24 19:21:09]

Done.

+  }
+}
+
+}  // namespace
+
+void PostTaskHelper(scoped_ptr<Task> task,
+                    scoped_refptr<Sequence> sequence,
+                    PriorityQueue* priority_queue,
+                    TaskTracker* task_tracker) {
+  DCHECK(task.get());

[gab, 2016/03/21 19:11:53]

No .get() for bool operations here and below

[fdoray, 2016/03/24 19:21:09]

Done.

+  DCHECK(sequence.get());
+  DCHECK(priority_queue);
+  DCHECK(task_tracker);
+
+  task_tracker->PostTask(Bind(&PostTaskCallback, sequence, priority_queue),

[gab, 2016/03/21 19:11:53]

std::move(sequence)

[fdoray, 2016/03/24 19:21:09]

Done.

+                         std::move(task));
+}
+
+}  // namespace internal
+}  // namespace base