Initial error handling code

src/core/SkOrderedReadBuffer.cpp

Index: src/core/SkOrderedReadBuffer.cpp
diff --git a/src/core/SkOrderedReadBuffer.cpp b/src/core/SkOrderedReadBuffer.cpp
index 9d991949084e5b9586d57baeaadf72363eb3e3ef..2043d030fa882a98431846461e2851652ddf1acc 100644
--- a/src/core/SkOrderedReadBuffer.cpp
+++ b/src/core/SkOrderedReadBuffer.cpp
@@ -268,7 +268,7 @@ SkTypeface* SkOrderedReadBuffer::readTypeface() {
     }
 }
 
-SkFlattenable* SkOrderedReadBuffer::readFlattenable() {
+SkFlattenable* SkOrderedReadBuffer::readFlattenable(SkFlattenable::Type) {

[mtklein, 2013/09/24 22:52:18]

A little // We ignore this Type parameter here because we're assuming the data
in the SkOrderedReadBuffer is trusted and we don't need to do type checking ?

Now that I'm thinking about it, I'll have to remember to come back through here
and _do_ use the Type for type checking in Debug builds, at least where
possible.

[sugoi1, 2013/09/25 21:15:27]

Done.

     SkFlattenable::Factory factory = NULL;
 
     if (fFactoryCount > 0) {