Enabling validation code in serialization and adding serialization to fuzzer

samplecode/SampleFilterFuzz.cpp

-
 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 #include "SampleCode.h"
 #include "SkBicubicImageFilter.h"
+#include "SkBitmapDevice.h"
 #include "SkBitmapSource.h"
 #include "SkBlurImageFilter.h"
 #include "SkCanvas.h"
 #include "SkColorFilter.h"
 #include "SkColorFilterImageFilter.h"
 #include "SkComposeImageFilter.h"
-#include "SkBitmapDevice.h"
+#include "SkData.h"
 #include "SkDisplacementMapEffect.h"
 #include "SkDropShadowImageFilter.h"
 #include "SkFlattenableSerialization.h"
 #include "SkLightingImageFilter.h"
 #include "SkMagnifierImageFilter.h"
 #include "SkMergeImageFilter.h"
 #include "SkMorphologyImageFilter.h"
 #include "SkOffsetImageFilter.h"
 #include "SkPerlinNoiseShader.h"
 #include "SkRandom.h"
@@ skipping 228 matching lines @@
         filter = new SkDisplacementMapEffect(make_channel_selector_type(),
                      make_channel_selector_type(), make_scalar(),
                      make_image_filter(false), make_image_filter());
         break;
     default:
         break;
     }
     return (filter || canBeNull) ? filter : make_image_filter(canBeNull);
 }
 
+SkImageFilter* make_serialized_image_filter() {
+    SkImageFilter* filter = make_image_filter(false);

[mtklein, 2013/10/25 16:17:31]

Coming new at this, sorry if these are dumb questions.  Where do this filter and
data get deleted?

[Stephen White, 2013/10/25 16:36:21]

Oooh, good catch. This probably leaks (Alexis can correct me if I'm wrong).

There are two Skia-approved ways to handle this:

- put the result of all new or factory functions into an SkAutoTUnref<Foo> at
the callsite below (my preference)
- call paint.setImageFilter(foo)->unref() to free the ref created by
new/factoryfn

(Grumble grumble need smart pointers grumble.)

[sugoi1, 2013/10/25 17:15:04]

I probably had an initial implementation that gave the ownership of these to
another object, but I changed this code so many times for tests, that I probably
removed all ownership of these at some point :) I added the SkAutoTUnref "smart"
pointers.

+    SkData* data = SkValidatingSerializeFlattenable(filter);
+    unsigned char* ptr = static_cast<unsigned char*>(const_cast<void*>(data->data()));

[mtklein, 2013/10/25 16:17:31]

Seems like you don't need the const_cast until you assign to p?

[sugoi1, 2013/10/25 17:15:04]

Done.

+    size_t len = data->size();
+#ifdef SK_ADD_RANDOM_BIT_FLIPS
+    unsigned char* p = ptr;
+    for (size_t i = 0; i < len; ++i, ++p) {
+        if ((R(1000) == 1)) { // 0.1% of the time, flip a bit
+            *p ^= (1 << R(8));
+        }
+    }
+#endif // SK_ADD_RANDOM_BIT_FLIPS
+    SkFlattenable* flattenable = SkValidatingDeserializeFlattenable(ptr, len,
+                                    SkImageFilter::GetFlattenableType());
+    SkASSERT(NULL != flattenable);
+    return static_cast<SkImageFilter*>(flattenable);
+}
+
 static void drawClippedBitmap(SkCanvas* canvas, int x, int y, const SkPaint& paint) {
     canvas->save();
     canvas->clipRect(SkRect::MakeXYWH(SkIntToScalar(x), SkIntToScalar(y),
         SkIntToScalar(kBitmapSize), SkIntToScalar(kBitmapSize)));
     canvas->drawBitmap(make_bitmap(), SkIntToScalar(x), SkIntToScalar(y), &paint);
     canvas->restore();
 }
 
 static void do_fuzz(SkCanvas* canvas) {
     SkPaint paint;
-    paint.setImageFilter(make_image_filter());
+    paint.setImageFilter(make_serialized_image_filter());
     drawClippedBitmap(canvas, 0, 0, paint);
 }
 
 //////////////////////////////////////////////////////////////////////////////
 
 class ImageFilterFuzzView : public SampleView {
 public:
     ImageFilterFuzzView() {
         this->setBGColor(0xFFDDDDDD);
     }
@@ skipping 18 matching lines @@
     }
 
 private:
     typedef SkView INHERITED;
 };
 
 //////////////////////////////////////////////////////////////////////////////
 
 static SkView* MyFactory() { return new ImageFilterFuzzView; }
 static SkViewRegister reg(MyFactory);