Trying to be consistent when members are allowed to be NULL

src/effects/SkColorFilterImageFilter.cpp

Index: src/effects/SkColorFilterImageFilter.cpp
diff --git a/src/effects/SkColorFilterImageFilter.cpp b/src/effects/SkColorFilterImageFilter.cpp
index 2a8df5c672109a4aee428e9801af6c305c0c89d4..170a0c6aa642ee7b71121f75e6c8ff89bf4b2d2b 100755
--- a/src/effects/SkColorFilterImageFilter.cpp
+++ b/src/effects/SkColorFilterImageFilter.cpp
@@ -60,6 +60,9 @@ bool matrix_needs_clamping(SkScalar matrix[20]) {
 SkColorFilterImageFilter* SkColorFilterImageFilter::Create(SkColorFilter* cf,
         SkImageFilter* input, const CropRect* cropRect, uint32_t uniqueID) {
     SkASSERT(cf);
+    if (NULL == cf) {
+        return NULL;
+    }
     SkScalar colorMatrix[20], inputMatrix[20];
     SkColorFilter* inputColorFilter;
     if (input && cf->asColorMatrix(colorMatrix)
@@ -79,8 +82,8 @@ SkColorFilterImageFilter* SkColorFilterImageFilter::Create(SkColorFilter* cf,
 SkColorFilterImageFilter::SkColorFilterImageFilter(SkColorFilter* cf,
         SkImageFilter* input, const CropRect* cropRect, uint32_t uniqueID)
     : INHERITED(1, &input, cropRect, uniqueID), fColorFilter(cf) {

[reed1, 2014/10/29 19:55:00]

suggestion nit: could just say fColorFilter(SkRef(cf)) { ...

-    SkASSERT(cf);
-    SkSafeRef(cf);
+    SkASSERT(fColorFilter);
+    fColorFilter->ref();
 }
 
 #ifdef SK_SUPPORT_LEGACY_DEEPFLATTENING
@@ -102,7 +105,7 @@ void SkColorFilterImageFilter::flatten(SkWriteBuffer& buffer) const {
 }
 
 SkColorFilterImageFilter::~SkColorFilterImageFilter() {
-    SkSafeUnref(fColorFilter);
+    fColorFilter->unref();
 }
 
 bool SkColorFilterImageFilter::onFilterImage(Proxy* proxy, const SkBitmap& source,