remove unused (by clients) SkUnitMapper

src/core/SkValidatingReadBuffer.cpp

Index: src/core/SkValidatingReadBuffer.cpp
diff --git a/src/core/SkValidatingReadBuffer.cpp b/src/core/SkValidatingReadBuffer.cpp
index 0112f18ec44d5b922d9a0e4455e6975b371577df..96f20a24833b7e23b7e34d54aa58fa28f45a848d 100644
--- a/src/core/SkValidatingReadBuffer.cpp
+++ b/src/core/SkValidatingReadBuffer.cpp
@@ -273,3 +273,15 @@ SkFlattenable* SkValidatingReadBuffer::readFlattenable(SkFlattenable::Type type)
     }
     return obj;
 }
+
+void SkValidatingReadBuffer::skipFlattenable() {
+    SkString name;
+    this->readString(&name);
+    if (fError) {
+        return;
+    }
+    uint32_t sizeRecorded = this->readUInt();

[scroggo, 2014/05/15 17:18:13]

There are a couple of cases where readFlattenable might have returned before
reaching its own call to readUInt(). Is it okay that this function calls it
without doing those checks?

[reed1, 2014/05/16 12:26:56]

1. Some of those tests involve knowing the type (enum), which I don't here
2. I think perhaps that logic is faulty, as it won't actually leave the buffer
past the data when it returns early.

I will file a bug/investigation for #2 separate from this CL.

+    this->skip(sizeRecorded);
+}
+
+