For BUILD.public, let SkPreConfig.h set SK_RELEASE based on NDEBUG.

include/core/SkPostConfig.h

 /*
  * Copyright 2006 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 // IWYU pragma: private, include "SkTypes.h"
 
 #ifndef SkPostConfig_DEFINED
@@ skipping 130 matching lines @@
 #  ifndef SK_A32_SHIFT
 #    define SK_A32_SHIFT 24
 #    define SK_R32_SHIFT 16
 #    define SK_G32_SHIFT 8
 #    define SK_B32_SHIFT 0
 #  endif
 #
 #endif
 
 #ifndef SK_ALWAYSBREAK
-#  ifdef SK_DEBUG
+#  if defined(GOOGLE3)
+#    include "base/examine_stack.h"
 #    define SK_ALWAYSBREAK(cond) do { \
               if (cond) break; \
               SkNO_RETURN_HINT(); \
+              SkDebugf("%s:%d: failed assertion \"%s\"\n", __FILE__, __LINE__, #cond); \
+              DumpStackTrace(0, DebugWriteToStderr, NULL); \

[mtklein, 2015/10/28 19:09:07]

Think we can just forward-declare this instead of including base/examine_stack.h
to avoid polluting the global namespace with `base`?

+              SK_CRASH(); \
+        } while (false)
+#  elif defined(SK_DEBUG)
+#    define SK_ALWAYSBREAK(cond) do { \
+              if (cond) break; \
+              SkNO_RETURN_HINT(); \
               SkDebugf("%s:%d: failed assertion \"%s\"\n", __FILE__, __LINE__, #cond); \
               SK_CRASH(); \
         } while (false)
 #  else
 #    define SK_ALWAYSBREAK(cond) do { if (cond) break; SK_CRASH(); } while (false)
 #  endif
 #endif
 
 /**
  *  We check to see if the SHIFT value has already been defined.
@@ skipping 187 matching lines @@
 #  define SK_GAMMA_EXPONENT (2.2f)
 #endif
 
 //////////////////////////////////////////////////////////////////////
 
 #ifndef GR_TEST_UTILS
 #  define GR_TEST_UTILS 1
 #endif
 
 #endif // SkPostConfig_DEFINED