TaskScheduler [5/9] Task Tracker

base/task_scheduler/task_traits.h

Index: base/task_scheduler/task_traits.h
diff --git a/base/task_scheduler/task_traits.h b/base/task_scheduler/task_traits.h
index 5bdfd2c4501ae6a0a8eddf1c74fb042157b491d9..98a6a4c8365c71cb1c8751a217e6d53c92c1c735 100644
--- a/base/task_scheduler/task_traits.h
+++ b/base/task_scheduler/task_traits.h
@@ -136,6 +136,11 @@ enum class ExecutionMode {
 // Pretty Printer for Google Test.
 void BASE_EXPORT PrintTo(const TaskPriority& task_priority, std::ostream* os);
 
+// Stream operator so TaskPriority can be used in assertion statements.

[gab, 2016/03/17 01:42:13]

s/TaskPriority/TaskShutdownBehavior/

(I still don't understand why this is operator<< and TaskPriority is PrintTo()?
Are they not asserted the same way?)

[fdoray, 2016/03/17 20:12:16]

TaskPriority is used in Google Test (EXPECT_EQ) while TaskShutdownBehavior is
used in a DCHECK_NE. Google Test supports both PrintTo and operator<< while
DCHECK supports operator<< only. I changed the code to use operator<< all the
time.

[gab, 2016/03/18 18:48:17]

Got it thanks, the now more consistent approach is less confusing :-).

+BASE_EXPORT std::ostream& operator<<(
+    std::ostream& os,
+    const TaskShutdownBehavior& shutdown_behavior);
+
 }  // namespace base
 
 #endif  // BASE_TASK_SCHEDULER_TASK_TRAITS_H_