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

Unified Diff: Source/platform/TaskSynchronizer.h

Issue 397733004: Allow assertions to be enabled in Blink Release builds. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebased. Created 6 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
« no previous file with comments | « Source/platform/LayoutUnit.h ('k') | Source/platform/TaskSynchronizer.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/platform/TaskSynchronizer.h
diff --git a/Source/platform/TaskSynchronizer.h b/Source/platform/TaskSynchronizer.h
index 1949498e5ae5ecc540b5606f158820c7d85a12b8..402587d32b1ab5730b5a4da51efc4358b28f59de 100644
--- a/Source/platform/TaskSynchronizer.h
+++ b/Source/platform/TaskSynchronizer.h
@@ -48,7 +48,7 @@ public:
// Called from a thread that executes the task.
void taskCompleted();
-#ifndef NDEBUG
+#if ENABLE(ASSERT)
bool hasCheckedForTermination() const { return m_hasCheckedForTermination; }
void setHasCheckedForTermination() { m_hasCheckedForTermination = true; }
#endif
@@ -59,7 +59,7 @@ private:
bool m_taskCompleted;
Mutex m_synchronousMutex;
ThreadCondition m_synchronousCondition;
-#ifndef NDEBUG
+#if ENABLE(ASSERT)
bool m_hasCheckedForTermination;
#endif
};
« no previous file with comments | « Source/platform/LayoutUnit.h ('k') | Source/platform/TaskSynchronizer.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698