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

src/utils/SkPictureUtils.cpp

 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkPictureUtils.h"
 #include "SkCanvas.h"
 #include "SkData.h"
@@ skipping 19 matching lines @@
 //            SkDebugf("--- already have %x %d\n", pr, genID);
         }
     }
 
 private:
     SkTDArray<SkPixelRef*>* fArray;
     SkTDArray<uint32_t>     fGenID;
 };
 
 static void not_supported() {
-    SkASSERT(!"this method should never be called");
+    SkDEBUGFAIL("this method should never be called");
 }
 
 static void nothing_to_do() {}
 
 /**
  *  This device will route all bitmaps (primitives and in shaders) to its PRSet.
  *  It should never actually draw anything, so there need not be any pixels
  *  behind its device-bitmap.
  */
 class GatherPixelRefDevice : public SkDevice {
@@ skipping 165 matching lines @@
     canvas.clipRect(area, SkRegion::kIntersect_Op, false);
     canvas.drawPicture(*pict);
 
     SkData* data = NULL;
     int count = array.count();
     if (count > 0) {
         data = SkData::NewFromMalloc(array.detach(), count * sizeof(SkPixelRef*));
     }
     return data;
 }