"NULL !=" = NULL

tests/DeferredCanvasTest.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 "../src/image/SkImagePriv.h"
 #include "../src/image/SkSurface_Base.h"
 #include "SkBitmap.h"
@@ skipping 663 matching lines @@
         return surface->getCanvas()->internal_private_accessTopLayerRenderTarget()->asTexture();
     } else
 #endif
     {
         return surface->peekPixels(NULL, NULL);
     }
 }
 
 static void TestDeferredCanvasSurface(skiatest::Reporter* reporter, GrContextFactory* factory) {
     SkImageInfo imageSpec = SkImageInfo::MakeN32Premul(10, 10);
-    bool useGpu = NULL != factory;
+    bool useGpu = SkToBool(factory);
     int cnt;
 #if SK_SUPPORT_GPU
     if (useGpu) {
         cnt = GrContextFactory::kGLContextTypeCnt;
     } else {
         cnt = 1;
     }
 #else
     SkASSERT(!useGpu);
     cnt = 1;
@@ skipping 10 matching lines @@
             if (NULL == context) {
                 return;
             }
 
             surface = SkSurface::NewRenderTarget(context, imageSpec);
         } else
 #endif
         {
            surface = SkSurface::NewRaster(imageSpec);
         }
-        SkASSERT(NULL != surface);
+        SkASSERT(surface);
         SkAutoTUnref<SkSurface> aur(surface);
         SkAutoTUnref<SkDeferredCanvas> canvas(SkDeferredCanvas::Create(surface));
 
         SkImage* image1 = canvas->newImageSnapshot();
         SkAutoTUnref<SkImage> aur_i1(image1);
         PixelPtr pixels1 = get_surface_ptr(surface, useGpu);
         // The following clear would normally trigger a copy on write, but
         // it won't because rendering is deferred.
         canvas->clear(SK_ColorBLACK);
         // Obtaining a snapshot directly from the surface (as opposed to the
@@ skipping 31 matching lines @@
         canvas->flush();
         PixelPtr pixels5 = get_surface_ptr(surface, useGpu);
         REPORTER_ASSERT(reporter, pixels4 == pixels5);
     }
 }
 
 static void TestDeferredCanvasSetSurface(skiatest::Reporter* reporter, GrContextFactory* factory) {
     SkImageInfo imageSpec = SkImageInfo::MakeN32Premul(10, 10);
     SkSurface* surface;
     SkSurface* alternateSurface;
-    bool useGpu = NULL != factory;
+    bool useGpu = SkToBool(factory);
     int cnt;
 #if SK_SUPPORT_GPU
     if (useGpu) {
         cnt = GrContextFactory::kGLContextTypeCnt;
     } else {
         cnt = 1;
     }
 #else
     SkASSERT(!useGpu);
     cnt = 1;
@@ skipping 11 matching lines @@
                 continue;
             }
             surface = SkSurface::NewRenderTarget(context, imageSpec);
             alternateSurface = SkSurface::NewRenderTarget(context, imageSpec);
         } else
 #endif
         {
             surface = SkSurface::NewRaster(imageSpec);
             alternateSurface = SkSurface::NewRaster(imageSpec);
         }
-        SkASSERT(NULL != surface);
-        SkASSERT(NULL != alternateSurface);
+        SkASSERT(surface);
+        SkASSERT(alternateSurface);
         SkAutoTUnref<SkSurface> aur1(surface);
         SkAutoTUnref<SkSurface> aur2(alternateSurface);
         PixelPtr pixels1 = get_surface_ptr(surface, useGpu);
         PixelPtr pixels2 = get_surface_ptr(alternateSurface, useGpu);
         SkAutoTUnref<SkDeferredCanvas> canvas(SkDeferredCanvas::Create(surface));
         SkAutoTUnref<SkImage> image1(canvas->newImageSnapshot());
         canvas->setSurface(alternateSurface);
         SkAutoTUnref<SkImage> image2(canvas->newImageSnapshot());
         REPORTER_ASSERT(reporter, image1->uniqueID() != image2->uniqueID());
         // Verify that none of the above operations triggered a surface copy on write.
@@ skipping 42 matching lines @@
     TestDeferredCanvasSurface(reporter, NULL);
     TestDeferredCanvasSetSurface(reporter, NULL);
 }
 
 DEF_GPUTEST(DeferredCanvas_GPU, reporter, factory) {
     if (factory != NULL) {
         TestDeferredCanvasSurface(reporter, factory);
         TestDeferredCanvasSetSurface(reporter, factory);
     }
 }