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-Commit-Position: refs/heads/master@{#8592}
git-svn-id: http://webrtc.googlecode.com/svn/trunk@8592 4adac7df-926f-26a2-2b94-8c16560cd09d
diff --git a/webrtc/base/checks.h b/webrtc/base/checks.h
index 823a310..5215868 100644
--- a/webrtc/base/checks.h
+++ b/webrtc/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