Adding error checks to SkRBuffer

include/core/SkFlattenableBuffers.h

Index: include/core/SkFlattenableBuffers.h
diff --git a/include/core/SkFlattenableBuffers.h b/include/core/SkFlattenableBuffers.h
index 8a94bb1db252d08d8c319e2705bc2b6bb2918344..8656f2f4974d26d76634a7ae56c0f02ac7a63a5c 100644
--- a/include/core/SkFlattenableBuffers.h
+++ b/include/core/SkFlattenableBuffers.h
@@ -144,7 +144,7 @@ public:
         return SkData::NewFromMalloc(buffer, len);
     }
 
-    virtual void validate(bool isValid) {}
+    virtual bool validate(bool isValid) { return true; }

[reed1, 2013/11/08 18:15:39]

Lets keep the impls for virtuals in the .cpp

[reed1, 2013/11/08 18:15:39]

I think we need some dox for this guy, now that there are 4 combinations to
interpret :

true <-- (true)
false <-- (true)
true <-- (false)
false <-- (false)

[sugoi1, 2013/11/08 18:40:56]

Done.

[sugoi1, 2013/11/08 18:40:56]

Done.

 
 private:
     template <typename T> T* readFlattenableT();