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

Unified Diff: base/thread_restrictions.cc

Issue 3824006: base: add a thread-safety assertion checker (Closed)
Patch Set: fix function name Created 10 years, 2 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
« base/thread_restrictions.h ('K') | « base/thread_restrictions.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/thread_restrictions.cc
diff --git a/base/thread_restrictions.cc b/base/thread_restrictions.cc
new file mode 100644
index 0000000000000000000000000000000000000000..5be8ad7133d390341fc1573c54efd64a2b591724
--- /dev/null
+++ b/base/thread_restrictions.cc
@@ -0,0 +1,42 @@
+// 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.
+
+#include "base/thread_restrictions.h"
+
+// This entire file is compiled out in Release mode.
+#ifndef NDEBUG
+
+#include "base/lazy_instance.h"
+#include "base/logging.h"
+#include "base/thread_local.h"
+
+namespace {
+
+static base::LazyInstance<base::ThreadLocalBoolean>
+ g_io_disallowed(base::LINKER_INITIALIZED);
+
+} // anonymous namespace
+
+namespace base {
+
+// static
+void ThreadRestrictions::SetIOAllowed(bool allowed) {
+ g_io_disallowed.Get().Set(!allowed);
+}
+
+// static
+void ThreadRestrictions::AssertIOAllowed() {
+ if (g_io_disallowed.Get().Get()) {
+ LOG(FATAL) <<
+ "Function marked as IO-only was called from a thread that "
+ "disallows IO! If this thread really should be allowed to "
+ "make IO calls, adjust the call to "
+ "base::ThreadRestrictions::SetIOAllowed() in this thread's "
+ "startup.";
+ }
+}
+
+} // namespace base
+
+#endif // NDEBUG
« base/thread_restrictions.h ('K') | « base/thread_restrictions.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698