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

src/images/SkImageDecoder.cpp

 /*
  * 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.
  */
 
 
 #include "SkImageDecoder.h"
 #include "SkBitmap.h"
@@ skipping 76 matching lines @@
             return "ICO";
         case kJPEG_Format:
             return "JPEG";
         case kPNG_Format:
             return "PNG";
         case kWBMP_Format:
             return "WBMP";
         case kWEBP_Format:
             return "WEBP";
         default:
-            SkASSERT(!"Invalid format type!");
+            SkDEBUGFAIL("Invalid format type!");
     }
     return "Unknown Format";
 }
 
 SkImageDecoder::Peeker* SkImageDecoder::setPeeker(Peeker* peeker) {
     SkRefCnt_SafeAssign(fPeeker, peeker);
     return peeker;
 }
 
 SkImageDecoder::Chooser* SkImageDecoder::setChooser(Chooser* chooser) {
@@ skipping 318 matching lines @@
     // Since Config is SkBitmap::kARGB_8888_Config, SkBitmapToImageInfo
     // will always succeed.
     SkAssertResult(SkBitmapToImageInfo(bm, info));
 
     if (NULL == target) {
         return true;
     }
 
     if (target->fRowBytes != SkToU32(bm.rowBytes())) {
         if (target->fRowBytes < SkImageMinRowBytes(*info)) {
-            SkASSERT(!"Desired row bytes is too small");
+            SkDEBUGFAIL("Desired row bytes is too small");
             return false;
         }
         bm.setConfig(bm.config(), bm.width(), bm.height(), target->fRowBytes);
     }
 
     // SkMemoryStream.rewind() will always return true.
     SkAssertResult(stream.rewind());
     return decode_pixels_to_8888(decoder.get(), &stream, &bm, target->fAddr);
 }
 
@@ skipping 13 matching lines @@
             if (kUnknown_Format == *format) {
                 if (stream->rewind()) {
                     *format = GetStreamFormat(stream);
                 }
             }
         }
         delete codec;
     }
     return success;
 }