Chromium Code Reviews| Index: src/images/SkImageDecoder_ktx.cpp |
| diff --git a/src/images/SkImageDecoder_ktx.cpp b/src/images/SkImageDecoder_ktx.cpp |
| index 539795ad9bb56b9ba4610ad6ae31d33fc7a8540b..156674565c0dff6c91adfed7331a02d16d558797 100644 |
| --- a/src/images/SkImageDecoder_ktx.cpp |
| +++ b/src/images/SkImageDecoder_ktx.cpp |
| @@ -50,12 +50,12 @@ private: |
| SkImageDecoder::Result SkKTXImageDecoder::onDecode(SkStream* stream, SkBitmap* bm, Mode mode) { |
| // TODO: Implement SkStream::copyToData() that's cheap for memory and file streams |
| - SkAutoDataUnref data(SkCopyStreamToData(stream)); |
| + auto data = SkCopyStreamToData(stream); |
| if (nullptr == data) { |
| return kFailure; |
| } |
| - SkKTXFile ktxFile(data); |
| + SkKTXFile ktxFile(data.get()); |
|
scroggo
2016/03/11 15:28:35
Should this class take an sk_sp?
reed1
2016/03/11 19:23:34
Yes (in a separate CL)
|
| if (!ktxFile.valid()) { |
| return kFailure; |
| } |