My clang now doesn't complain about !"foo".

src/utils/SkJSON.cpp

Index: src/utils/SkJSON.cpp
diff --git a/src/utils/SkJSON.cpp b/src/utils/SkJSON.cpp
index 9b122082021fee0727e659797e644cc471a991c9..1aedf58d8b6c1dec8369fea8885d2e4be28a80aa 100644
--- a/src/utils/SkJSON.cpp
+++ b/src/utils/SkJSON.cpp
@@ -414,7 +414,7 @@ void SkJSON::Object::dumpLevel(int level) const {
                 SkDebugf("%s", slot->fValue.fBool ? "true" : "false");
                 break;
             default:
-                SkASSERT(!"how did I get here");
+                SkDEBUGFAIL("how did I get here");
                 break;
         }
         if (slot->fNext) {
@@ -490,7 +490,7 @@ void SkJSON::Array::dumpLevel(int level) const {
             SkDebugf(" %s ", fArray.fInts[last] ? "true" : "false");
         } break;
         default:
-            SkASSERT(!"unsupported array type");
+            SkDEBUGFAIL("unsupported array type");
             break;
     }
 }