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

Unified Diff: base/sequence_checker.h

Issue 2165663003: TaskScheduler: Add SequenceToken and ScopedSetSequenceTokenForCurrentThread (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: self-review Created 4 years, 5 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/sequence_checker.h
diff --git a/base/sequence_checker.h b/base/sequence_checker.h
index ad0182825c24805548f05972c7c04e6813688bc7..b23941cdadf67594d09b1b9e988744507ac4cb68 100644
--- a/base/sequence_checker.h
+++ b/base/sequence_checker.h
@@ -29,10 +29,11 @@ class SequenceCheckerDoNothing {
void DetachFromSequence() {}
};
-// SequenceChecker is a helper class used to help verify that some
-// methods of a class are called in sequence -- that is, called from
-// the same SequencedTaskRunner. It is a generalization of
-// ThreadChecker; see comments in sequence_checker_impl.h for details.
+// SequenceChecker is a helper class to verify that calls to some methods of a
+// class are sequenced. Calls are sequenced when they are issued:
+// - From tasks posted to SequencedTaskRunners or SingleThreadTaskRunners bound
+// to the same sequence, or,
+// - From a single thread outside of any task.
gab 2016/07/26 20:49:48 I find this last statement confusing, MessageLoop
fdoray 2016/07/28 18:10:04 I think we don't plan to support child single-thre
gab 2016/07/29 13:31:37 On 2016/07/28 18:10:04, fdoray wrote:
//
// Example:
// class MyClass {
« no previous file with comments | « base/base.gypi ('k') | base/sequence_checker_impl.h » ('j') | base/sequence_checker_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698