remove unused (by clients) SkUnitMapper

src/core/SkReadBuffer.cpp

Index: src/core/SkReadBuffer.cpp
diff --git a/src/core/SkReadBuffer.cpp b/src/core/SkReadBuffer.cpp
index b4bc87529ab5a5b1e1e620d2775f7e7a5f1dedf1..f57fc1ada037982448f34648c553892e29348f8e 100644
--- a/src/core/SkReadBuffer.cpp
+++ b/src/core/SkReadBuffer.cpp
@@ -335,3 +335,21 @@ SkFlattenable* SkReadBuffer::readFlattenable(SkFlattenable::Type ft) {
     }
     return obj;
 }
+
+void SkReadBuffer::skipFlattenable() {
+    if (fFactoryCount > 0) {

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

This structure does appear to be the same as readFlattenable(), but it does
worry me they could get out of sync. Maybe we just need some comments?

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

Done.

+        if (0 == fReader.readU32()) {
+            return;
+        }
+    } else if (fFactoryTDArray) {
+        if (0 == fReader.readU32()) {
+            return;
+        }
+    } else {
+        if (NULL == this->readFunctionPtr()) {
+            return;
+        }
+    }
+    uint32_t sizeRecorded = fReader.readU32();
+    fReader.skip(sizeRecorded);
+}