commit 5dd45021c37e24cd2c8e91a0f1a1d28a77ad613c
author djsollen@google.com <djsollen@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> Thu Mar 21 13:30:54 2013 +0000
committer djsollen@google.com <djsollen@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> Thu Mar 21 13:30:54 2013 +0000
tree c0a6876cf6cc3a32bf99dad448a8dbbcc6f4a17c
parent 5dc4ca1ce6e89e4de9675ef44215333480a497d2

Update code to use helper function for better readability/searching.

Review URL: https://codereview.chromium.org/12834012

git-svn-id: http://skia.googlecode.com/svn/trunk@8287 2bbb7eff-a529-9590-31e7-b0007b416f81

src/images/SkImageDecoder_libbmp.cpp

diff --git a/src/images/SkImageDecoder_libbmp.cpp b/src/images/SkImageDecoder_libbmp.cpp
index 1dcea8a..488eddc 100644
--- a/src/images/SkImageDecoder_libbmp.cpp
+++ b/src/images/SkImageDecoder_libbmp.cpp
@@ -37,12 +37,11 @@
 static SkImageDecoder* sk_libbmp_dfactory(SkStream* stream) {
     static const char kBmpMagic[] = { 'B', 'M' };
 
-    size_t len = stream->getLength();
+
     char buffer[sizeof(kBmpMagic)];
 
-    if (len > sizeof(kBmpMagic) &&
-            stream->read(buffer, sizeof(kBmpMagic)) == sizeof(kBmpMagic) &&
-            !memcmp(buffer, kBmpMagic, sizeof(kBmpMagic))) {
+    if (stream->read(buffer, sizeof(kBmpMagic)) == sizeof(kBmpMagic) &&
+        !memcmp(buffer, kBmpMagic, sizeof(kBmpMagic))) {
         return SkNEW(SkBMPImageDecoder);
     }
     return NULL;

src/images/SkImageDecoder_libico.cpp

diff --git a/src/images/SkImageDecoder_libico.cpp b/src/images/SkImageDecoder_libico.cpp
index 14f575b..ffc59e0 100644
--- a/src/images/SkImageDecoder_libico.cpp
+++ b/src/images/SkImageDecoder_libico.cpp
@@ -75,7 +75,7 @@
 
 bool SkICOImageDecoder::onDecode(SkStream* stream, SkBitmap* bm, Mode mode)
 {
-    size_t length = stream->read(NULL, 0);
+    size_t length = stream->getLength();
     SkAutoMalloc autoMal(length);
     unsigned char* buf = (unsigned char*)autoMal.get();
     if (stream->read((void*)buf, length) != length) {

src/ports/SkFontHost_FreeType.cpp

diff --git a/src/ports/SkFontHost_FreeType.cpp b/src/ports/SkFontHost_FreeType.cpp
index b08aa8e..6a36f44 100644
--- a/src/ports/SkFontHost_FreeType.cpp
+++ b/src/ports/SkFontHost_FreeType.cpp
@@ -1333,7 +1333,7 @@
         args.memory_size = stream->getLength();
     } else {
         memset(&streamRec, 0, sizeof(streamRec));
-        streamRec.size = stream->read(NULL, 0);
+        streamRec.size = stream->getLength();
         streamRec.descriptor.pointer = stream;
         streamRec.read  = sk_stream_read;
         streamRec.close = sk_stream_close;

src/ports/SkXMLParser_expat.cpp

diff --git a/src/ports/SkXMLParser_expat.cpp b/src/ports/SkXMLParser_expat.cpp
index afc9f79..3eba2fe 100644
--- a/src/ports/SkXMLParser_expat.cpp
+++ b/src/ports/SkXMLParser_expat.cpp
@@ -113,7 +113,7 @@
 
 bool SkXMLParser::parse(SkStream& input)
 {
-    size_t          len = input.read(NULL, 0);
+    size_t          len = input.getLength();
     SkAutoMalloc    am(len);
     char*           doc = (char*)am.get();
 

src/ports/SkXMLParser_tinyxml.cpp

diff --git a/src/ports/SkXMLParser_tinyxml.cpp b/src/ports/SkXMLParser_tinyxml.cpp
index f357592..5f9f3a6 100644
--- a/src/ports/SkXMLParser_tinyxml.cpp
+++ b/src/ports/SkXMLParser_tinyxml.cpp
@@ -58,7 +58,7 @@
 
 bool SkXMLParser::parse(SkStream& stream)
 {
-    size_t size = stream.read(NULL, 0);
+    size_t size = stream.getLength();
 
     SkAutoMalloc    buffer(size + 1);
     char*           buf = (char*)buffer.get();

src/ports/SkXMLPullParser_expat.cpp

diff --git a/src/ports/SkXMLPullParser_expat.cpp b/src/ports/SkXMLPullParser_expat.cpp
index 1d1615b..44a3c7f 100644
--- a/src/ports/SkXMLPullParser_expat.cpp
+++ b/src/ports/SkXMLPullParser_expat.cpp
@@ -152,7 +152,7 @@
     XML_SetCharacterDataHandler(p, text_proc);
     XML_SetUserData(p, &fImpl->fData);
 
-    size_t len = fStream->read(NULL, 0);
+    size_t len = fStream->getLength();
     fImpl->fBufferLen = len;
     fImpl->fBuffer = sk_malloc_throw(len);
     fStream->rewind();