commit ab0b929321d37669165d5795268fa10a8c97ec5b
author kwiberg <kwiberg@webrtc.org> Mon Oct 03 05:04:21 2016 -0700
committer Commit bot <commit-bot@chromium.org> Mon Oct 03 12:04:25 2016 +0000
tree c09ee266d00be445969b340362c3bd84cca32a00
parent 1d477ea6d5221c6654db4c0a7cebc68bb70aa080

Test RTC_DCHECK_IS_ON instead of checking DCHECK_ALWAYS_ON everywhere

The former is always defined (by webrtc/base/checks.h) to either 0 or
1, whereas the latter isn't necessarily defined.

NOTRY=true
BUG=webrtc:6451

Review-Url: https://codereview.webrtc.org/2384693002
Cr-Commit-Position: refs/heads/master@{#14474}

webrtc/modules/utility/source/process_thread_impl.cc

diff --git a/webrtc/modules/utility/source/process_thread_impl.cc b/webrtc/modules/utility/source/process_thread_impl.cc
index 66534de..3332cd6 100644
--- a/webrtc/modules/utility/source/process_thread_impl.cc
+++ b/webrtc/modules/utility/source/process_thread_impl.cc
@@ -133,7 +133,7 @@
   RTC_DCHECK(thread_checker_.CalledOnValidThread());
   RTC_DCHECK(module);
 
-#if (!defined(NDEBUG) || defined(DCHECK_ALWAYS_ON))
+#if RTC_DCHECK_IS_ON
   {
     // Catch programmer error.
     rtc::CritScope lock(&lock_);