Always define RTC_NOTREACHED, not just in non-chromium builds

R=tommi@webrtc.org
TBR=tommi

Review URL: https://webrtc-codereview.appspot.com/40189004

Cr-Original-Commit-Position: refs/heads/master@{#8592}
Cr-Mirrored-From: https://chromium.googlesource.com/external/webrtc
Cr-Mirrored-Commit: 487afc704d5c6a2519f2decdb47130c7b3424992
diff --git a/base/checks.h b/base/checks.h
index 823a310..5215868 100644
--- a/base/checks.h
+++ b/base/checks.h
@@ -185,9 +185,6 @@
 #define DCHECK_GT(v1, v2) EAT_STREAM_PARAMETERS((v1) > (v2))
 #endif
 
-#define RTC_UNREACHABLE_CODE_HIT false
-#define RTC_NOTREACHED() DCHECK(RTC_UNREACHABLE_CODE_HIT)
-
 // This is identical to LogMessageVoidify but in name.
 class FatalMessageVoidify {
  public:
@@ -199,6 +196,9 @@
 
 #endif  // WEBRTC_CHROMIUM_BUILD
 
+#define RTC_UNREACHABLE_CODE_HIT false
+#define RTC_NOTREACHED() DCHECK(RTC_UNREACHABLE_CODE_HIT)
+
 #define FATAL() rtc::FatalMessage(__FILE__, __LINE__).stream()
 // TODO(ajm): Consider adding NOTIMPLEMENTED and NOTREACHED macros when
 // base/logging.h and system_wrappers/logging.h are consolidated such that we