Convert GrCrash->SkFAIL GrDebugCrash->SkDEBUGFAIL

include/core/SkTypes.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.
  */
 
 #ifndef SkTypes_DEFINED
 #define SkTypes_DEFINED
 
@@ skipping 95 matching lines @@
     #define SkDEBUGFAIL(message)
     #define SkDEBUGCODE(code)
     #define SkDEBUGF(args)
     #define SkDECLAREPARAM(type, var)
     #define SkPARAM(var)
 
     // unlike SkASSERT, this guy executes its condition in the non-debug build
     #define SkAssertResult(cond)        cond
 #endif
 
+#define SkFAIL(message)                 SK_DEBUGBREAK(false && message)

[mtklein, 2014/04/30 13:51:37]

Isn't this the same as SkDEBUGFAIL?

[reed1, 2014/04/30 14:01:28]

I think SkDEBUGFAIL does nothing in a release build.

[bsalomon, 2014/04/30 14:01:44]

SK_DEBUGBREAK is misnamed. It is really SK_ALWAYSBREAK.

https://code.google.com/p/skia/source/browse/trunk/include/core/SkPostConfig....

https://code.google.com/p/skia/source/browse/trunk/include/core/SkPostConfig....

+
 #ifdef SK_DEVELOPER
     #define SkDEVCODE(code)             code
 #else
     #define SkDEVCODE(code)
 #endif
 
 #ifdef SK_IGNORE_TO_STRING
     #define SK_TO_STRING_NONVIRT()
     #define SK_TO_STRING_VIRT()
     #define SK_TO_STRING_PUREVIRT()
@@ skipping 529 matching lines @@
 private:
     void*       fPtr;
     size_t      fSize;  // can be larger than the requested size (see kReuse)
     uint32_t    fStorage[(kSize + 3) >> 2];
 };
 // Can't guard the constructor because it's a template class.
 
 #endif /* C++ */
 
 #endif