Templetized SkWriter32 readTAt() & overwriteTAt()

src/core/SkPictureRecord.h

Index: src/core/SkPictureRecord.h
diff --git a/src/core/SkPictureRecord.h b/src/core/SkPictureRecord.h
index 6d14c621aafb510dd5e13a6f01fa50f866c19992..069671a40cd5cebc7c8fe3377b3ff7fca7123e1f 100644
--- a/src/core/SkPictureRecord.h
+++ b/src/core/SkPictureRecord.h
@@ -156,7 +156,7 @@ private:
         fWriter.writeInt(value);
     }
     void addScalar(SkScalar scalar) {
-        fWriter.writeScalar(scalar);
+        fWriter.writeT<SkScalar>(scalar);

[mtklein, 2014/02/06 21:38:16]

This may just be me, but when I see a call like this I assume we're giving the
type explicitly to force a type onto the argument.  I'd generally prefer to see
this as fWriter.writeT(scalar) for simple call sites, so that if we do see an
fWriter.writeT<SomeType>(x) we can know we've got some tricky reason to mention
the type.

[f(malita), 2014/02/10 16:10:35]

Will do.

     }
 
     void addBitmap(const SkBitmap& bitmap);