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

Unified Diff: base/task_scheduler/post_task.h

Issue 2628893005: Remove experimental comment on post_task.h and update comment about default traits. (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/task_scheduler/post_task.h
diff --git a/base/task_scheduler/post_task.h b/base/task_scheduler/post_task.h
index ee1de7fb421344197196ab0531d14d56d9966a40..1c5c7096b5c1e4e21ffdab0ff16ac4535de9fdc7 100644
--- a/base/task_scheduler/post_task.h
+++ b/base/task_scheduler/post_task.h
@@ -21,9 +21,6 @@ namespace base {
// This is the preferred interface to post tasks to the TaskScheduler.
//
-// Note: The TaskScheduler is still in an experimental phase in Chrome. Please
-// refrain from using this API unless you know what you are doing.
-//
// TaskScheduler must have been registered for the current process via
// TaskScheduler::SetInstance() before the functions below are valid.
//
@@ -52,8 +49,8 @@ namespace base {
// task_runner.PostTask(FROM_HERE, Bind(...));
//
// The default TaskTraits apply to tasks that:
-// (1) don't need to do I/O,
-// (2) don't affect user interaction and/or visible elements, and
+// (1) don't block (ref. MayBlock() and WithBaseSyncPrimitives()),
+// (2) prefer inheriting the current priority to specifying their own, and
// (3) can either block shutdown or be skipped on shutdown
// (barring current TaskScheduler default).
// If those loose requirements are sufficient for your task, use
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698