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

Unified Diff: base/task_scheduler/task_tracker.cc

Issue 2464963002: TaskScheduler: Remove base::ExecutionMode. (Closed)
Patch Set: self-review Created 4 years, 1 month 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/task_tracker.cc
diff --git a/base/task_scheduler/task_tracker.cc b/base/task_scheduler/task_tracker.cc
index 90c52937747ea459761ad8b972b6f4b2c6e2b8a7..fb50116adee4ab2f85d7d86006860a052453b31d 100644
--- a/base/task_scheduler/task_tracker.cc
+++ b/base/task_scheduler/task_tracker.cc
@@ -25,11 +25,15 @@ namespace internal {
namespace {
+constexpr char kParallelExecutionMode[] = "parallel";
+constexpr char kSequencedExecutionMode[] = "sequenced";
+constexpr char kSingleThreadExecutionMode[] = "single thread";
+
// An immutable copy of a scheduler task's info required by tracing.
class TaskTracingInfo : public trace_event::ConvertableToTraceFormat {
public:
TaskTracingInfo(const TaskTraits& task_traits,
- ExecutionMode execution_mode,
+ const char* execution_mode,
const SequenceToken& sequence_token)
: task_traits_(task_traits),
execution_mode_(execution_mode),
@@ -40,7 +44,7 @@ class TaskTracingInfo : public trace_event::ConvertableToTraceFormat {
private:
const TaskTraits task_traits_;
- const ExecutionMode execution_mode_;
+ const char* execution_mode_;
gab 2016/10/31 19:10:11 const char* const
fdoray 2016/10/31 19:44:52 Done.
const SequenceToken sequence_token_;
DISALLOW_COPY_AND_ASSIGN(TaskTracingInfo);
@@ -51,9 +55,8 @@ void TaskTracingInfo::AppendAsTraceFormat(std::string* out) const {
dict.SetString("task_priority",
base::TaskPriorityToString(task_traits_.priority()));
- dict.SetString("execution_mode",
- base::ExecutionModeToString(execution_mode_));
- if (execution_mode_ != ExecutionMode::PARALLEL)
+ dict.SetString("execution_mode", execution_mode_);
+ if (execution_mode_ != kParallelExecutionMode)
dict.SetInteger("sequence_token", sequence_token_.ToInternalValue());
std::string tmp;
@@ -243,11 +246,11 @@ bool TaskTracker::RunTask(std::unique_ptr<Task> task,
TRACE_TASK_EXECUTION(kRunFunctionName, *task);
- const ExecutionMode execution_mode =
+ const char* execution_mode =
gab 2016/10/31 19:10:11 const char* const
fdoray 2016/10/31 19:44:52 Done.
task->single_thread_task_runner_ref
- ? ExecutionMode::SINGLE_THREADED
- : (task->sequenced_task_runner_ref ? ExecutionMode::SEQUENCED
- : ExecutionMode::PARALLEL);
+ ? kSingleThreadExecutionMode
+ : (task->sequenced_task_runner_ref ? kSequencedExecutionMode
+ : kParallelExecutionMode);
// TODO(gab): In a better world this would be tacked on as an extra arg
// to the trace event generated above. This is not possible however until
// http://crbug.com/652692 is resolved.

Powered by Google App Engine
This is Rietveld 408576698