summaryrefslogtreecommitdiffstats
path: root/base/thread_checker.h
diff options
context:
space:
mode:
authorlevin@chromium.org <levin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-08-24 04:29:56 +0000
committerlevin@chromium.org <levin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-08-24 04:29:56 +0000
commitd9ddc963c18de9673711493052df08df3c838d00 (patch)
tree9bacde424cf1567d9aedc65d18ae50e93c133e1d /base/thread_checker.h
parent0b55debb0e1a686354fe461f0743f3835dc48b8f (diff)
downloadchromium_src-d9ddc963c18de9673711493052df08df3c838d00.zip
chromium_src-d9ddc963c18de9673711493052df08df3c838d00.tar.gz
chromium_src-d9ddc963c18de9673711493052df08df3c838d00.tar.bz2
Sometimes you want to enforce that some but not all of your methods are called on the same thread as the constructor. ThreadChecker allows you to do this.
BUG=38475 TEST=base_unittests --gtest_filter=NonThreadSafeTest.* and base_unittests --gtest_filter=ThreadChecker.* Review URL: http://codereview.chromium.org/3148032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57140 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/thread_checker.h')
-rw-r--r--base/thread_checker.h53
1 files changed, 53 insertions, 0 deletions
diff --git a/base/thread_checker.h b/base/thread_checker.h
new file mode 100644
index 0000000..4e1d792
--- /dev/null
+++ b/base/thread_checker.h
@@ -0,0 +1,53 @@
+// Copyright (c) 2010 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.
+
+#ifndef BASE_THREAD_CHECKER_H_
+#define BASE_THREAD_CHECKER_H_
+#pragma once
+
+#include "base/platform_thread.h"
+
+// Before using this class, please consider using NonThreadSafe as it
+// makes it much easier to determine the nature of your class.
+//
+// A helper class used to help verify that some methods of a class are
+// called from the same thread. One can inherit from this class and use
+// CalledOnValidThread() to verify.
+//
+// Inheriting from class indicates that one must be careful when using the class
+// with multiple threads. However, it is up to the class document to indicate
+// how it can be used with threads.
+//
+// Example:
+// class MyClass : public ThreadChecker {
+// public:
+// void Foo() {
+// DCHECK(CalledOnValidThread());
+// ... (do stuff) ...
+// }
+// }
+//
+// In Release mode, CalledOnValidThread will always return true.
+//
+#ifndef NDEBUG
+class ThreadChecker {
+ public:
+ ThreadChecker();
+
+ bool CalledOnValidThread() const;
+
+ private:
+ const PlatformThreadId valid_thread_id_;
+};
+#else
+// Do nothing in release mode.
+class ThreadChecker {
+ public:
+ bool CalledOnValidThread() const {
+ return true;
+ }
+};
+#endif // NDEBUG
+
+#endif // BASE_THREAD_CHECKER_H_