Chromium Code Reviews| Index: src/codec/SkCodec.cpp |
| diff --git a/src/codec/SkCodec.cpp b/src/codec/SkCodec.cpp |
| index 87a7ded7dd6bd2e0c0a6da19de1d9782a50553c3..0736f2a2691698180399971b3337aeccec5fc050 100644 |
| --- a/src/codec/SkCodec.cpp |
| +++ b/src/codec/SkCodec.cpp |
| @@ -62,13 +62,16 @@ SkCodec* SkCodec::NewFromStream(SkStream* stream, |
| // we trust the caller to use a large enough buffer. |
| if (0 == bytesRead) { |
| - SkCodecPrintf("Could not peek!\n"); |
| + // TODO: After implementing peek in CreateJavaOutputStreamAdaptor.cpp, this |
| + // printf could be useful to notice failures. |
| + // SkCodecPrintf("Encoded image data failed to peek!\n"); |
| + |
| // It is possible the stream does not support peeking, but does support |
| // rewinding. |
| // Attempt to read() and pass the actual amount read to the decoder. |
| bytesRead = stream->read(buffer, bytesToRead); |
| if (!stream->rewind()) { |
| - SkCodecPrintf("Could not rewind!\n"); |
| + SkCodecPrintf("Encoded image data could not peek or rewind to determine format!\n"); |
|
scroggo
2016/01/06 16:57:52
I could imagine making SkCodecPrintf add something
msarett
2016/01/06 17:11:27
Yes I agree.
|
| return nullptr; |
| } |
| } |