epoger@google.com | ec3ed6a | 2011-07-28 14:26:00 +0000 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2006 The Android Open Source Project |
| 3 | * |
| 4 | * Use of this source code is governed by a BSD-style license that can be |
| 5 | * found in the LICENSE file. |
| 6 | */ |
| 7 | |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 8 | |
| 9 | #include "SkImageDecoder.h" |
| 10 | #include "SkBitmap.h" |
scroggo@google.com | f8d7d27 | 2013-02-22 21:38:35 +0000 | [diff] [blame] | 11 | #include "SkImagePriv.h" |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 12 | #include "SkPixelRef.h" |
| 13 | #include "SkStream.h" |
| 14 | #include "SkTemplates.h" |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 15 | #include "SkCanvas.h" |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 16 | |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 17 | SkImageDecoder::SkImageDecoder() |
scroggo@google.com | 7e6fcee | 2013-05-03 20:14:28 +0000 | [diff] [blame] | 18 | : fPeeker(NULL) |
reed | 5926b86 | 2014-06-11 10:33:13 -0700 | [diff] [blame] | 19 | #ifdef SK_SUPPORT_LEGACY_IMAGEDECODER_CHOOSER |
scroggo@google.com | 7e6fcee | 2013-05-03 20:14:28 +0000 | [diff] [blame] | 20 | , fChooser(NULL) |
reed | 5926b86 | 2014-06-11 10:33:13 -0700 | [diff] [blame] | 21 | #endif |
scroggo@google.com | 7e6fcee | 2013-05-03 20:14:28 +0000 | [diff] [blame] | 22 | , fAllocator(NULL) |
| 23 | , fSampleSize(1) |
reed | bfefc7c | 2014-06-12 17:40:00 -0700 | [diff] [blame] | 24 | , fDefaultPref(kUnknown_SkColorType) |
reed | be08ace | 2014-07-08 11:15:59 -0700 | [diff] [blame] | 25 | , fPreserveSrcDepth(false) |
scroggo@google.com | 7e6fcee | 2013-05-03 20:14:28 +0000 | [diff] [blame] | 26 | , fDitherImage(true) |
scroggo@google.com | 8d23924 | 2013-10-01 17:27:15 +0000 | [diff] [blame] | 27 | , fSkipWritingZeroes(false) |
scroggo@google.com | 2bbc2c9 | 2013-06-14 15:33:20 +0000 | [diff] [blame] | 28 | , fPreferQualityOverSpeed(false) |
| 29 | , fRequireUnpremultipliedColors(false) { |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 30 | } |
| 31 | |
| 32 | SkImageDecoder::~SkImageDecoder() { |
reed@google.com | 82065d6 | 2011-02-07 15:30:46 +0000 | [diff] [blame] | 33 | SkSafeUnref(fPeeker); |
reed | 5926b86 | 2014-06-11 10:33:13 -0700 | [diff] [blame] | 34 | #ifdef SK_SUPPORT_LEGACY_IMAGEDECODER_CHOOSER |
reed@google.com | 82065d6 | 2011-02-07 15:30:46 +0000 | [diff] [blame] | 35 | SkSafeUnref(fChooser); |
reed | 5926b86 | 2014-06-11 10:33:13 -0700 | [diff] [blame] | 36 | #endif |
reed@google.com | 82065d6 | 2011-02-07 15:30:46 +0000 | [diff] [blame] | 37 | SkSafeUnref(fAllocator); |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 38 | } |
| 39 | |
scroggo@google.com | 468142b | 2013-07-09 15:48:24 +0000 | [diff] [blame] | 40 | void SkImageDecoder::copyFieldsToOther(SkImageDecoder* other) { |
| 41 | if (NULL == other) { |
| 42 | return; |
| 43 | } |
| 44 | other->setPeeker(fPeeker); |
reed | 5926b86 | 2014-06-11 10:33:13 -0700 | [diff] [blame] | 45 | #ifdef SK_SUPPORT_LEGACY_IMAGEDECODER_CHOOSER |
scroggo@google.com | 468142b | 2013-07-09 15:48:24 +0000 | [diff] [blame] | 46 | other->setChooser(fChooser); |
reed | 5926b86 | 2014-06-11 10:33:13 -0700 | [diff] [blame] | 47 | #endif |
scroggo@google.com | 468142b | 2013-07-09 15:48:24 +0000 | [diff] [blame] | 48 | other->setAllocator(fAllocator); |
| 49 | other->setSampleSize(fSampleSize); |
reed | be08ace | 2014-07-08 11:15:59 -0700 | [diff] [blame] | 50 | other->setPreserveSrcDepth(fPreserveSrcDepth); |
scroggo@google.com | 8d23924 | 2013-10-01 17:27:15 +0000 | [diff] [blame] | 51 | other->setDitherImage(fDitherImage); |
| 52 | other->setSkipWritingZeroes(fSkipWritingZeroes); |
scroggo@google.com | 468142b | 2013-07-09 15:48:24 +0000 | [diff] [blame] | 53 | other->setPreferQualityOverSpeed(fPreferQualityOverSpeed); |
| 54 | other->setRequireUnpremultipliedColors(fRequireUnpremultipliedColors); |
| 55 | } |
| 56 | |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 57 | SkImageDecoder::Format SkImageDecoder::getFormat() const { |
| 58 | return kUnknown_Format; |
| 59 | } |
| 60 | |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 61 | const char* SkImageDecoder::getFormatName() const { |
scroggo@google.com | f98118e | 2013-05-15 14:53:49 +0000 | [diff] [blame] | 62 | return GetFormatName(this->getFormat()); |
| 63 | } |
| 64 | |
| 65 | const char* SkImageDecoder::GetFormatName(Format format) { |
| 66 | switch (format) { |
scroggo@google.com | 39edf4c | 2013-04-25 17:33:51 +0000 | [diff] [blame] | 67 | case kUnknown_Format: |
| 68 | return "Unknown Format"; |
| 69 | case kBMP_Format: |
| 70 | return "BMP"; |
| 71 | case kGIF_Format: |
| 72 | return "GIF"; |
| 73 | case kICO_Format: |
| 74 | return "ICO"; |
robertphillips@google.com | 8cf81e0 | 2014-05-22 18:40:29 +0000 | [diff] [blame] | 75 | case kPKM_Format: |
| 76 | return "PKM"; |
krajcevski | 99ffe24 | 2014-06-03 13:04:35 -0700 | [diff] [blame] | 77 | case kKTX_Format: |
| 78 | return "KTX"; |
krajcevski | 95b1b3d | 2014-08-07 12:58:38 -0700 | [diff] [blame] | 79 | case kASTC_Format: |
| 80 | return "ASTC"; |
scroggo@google.com | 39edf4c | 2013-04-25 17:33:51 +0000 | [diff] [blame] | 81 | case kJPEG_Format: |
| 82 | return "JPEG"; |
| 83 | case kPNG_Format: |
| 84 | return "PNG"; |
| 85 | case kWBMP_Format: |
| 86 | return "WBMP"; |
| 87 | case kWEBP_Format: |
| 88 | return "WEBP"; |
| 89 | default: |
mtklein@google.com | 330313a | 2013-08-22 15:37:26 +0000 | [diff] [blame] | 90 | SkDEBUGFAIL("Invalid format type!"); |
scroggo@google.com | 39edf4c | 2013-04-25 17:33:51 +0000 | [diff] [blame] | 91 | } |
| 92 | return "Unknown Format"; |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 93 | } |
| 94 | |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 95 | SkImageDecoder::Peeker* SkImageDecoder::setPeeker(Peeker* peeker) { |
| 96 | SkRefCnt_SafeAssign(fPeeker, peeker); |
| 97 | return peeker; |
| 98 | } |
| 99 | |
reed | 5926b86 | 2014-06-11 10:33:13 -0700 | [diff] [blame] | 100 | #ifdef SK_SUPPORT_LEGACY_IMAGEDECODER_CHOOSER |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 101 | SkImageDecoder::Chooser* SkImageDecoder::setChooser(Chooser* chooser) { |
| 102 | SkRefCnt_SafeAssign(fChooser, chooser); |
| 103 | return chooser; |
| 104 | } |
reed | 5926b86 | 2014-06-11 10:33:13 -0700 | [diff] [blame] | 105 | #endif |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 106 | |
| 107 | SkBitmap::Allocator* SkImageDecoder::setAllocator(SkBitmap::Allocator* alloc) { |
| 108 | SkRefCnt_SafeAssign(fAllocator, alloc); |
| 109 | return alloc; |
| 110 | } |
| 111 | |
| 112 | void SkImageDecoder::setSampleSize(int size) { |
| 113 | if (size < 1) { |
| 114 | size = 1; |
| 115 | } |
| 116 | fSampleSize = size; |
| 117 | } |
| 118 | |
reed | 5926b86 | 2014-06-11 10:33:13 -0700 | [diff] [blame] | 119 | #ifdef SK_SUPPORT_LEGACY_IMAGEDECODER_CHOOSER |
reed | 6c22573 | 2014-06-09 19:52:07 -0700 | [diff] [blame] | 120 | // TODO: change Chooser virtual to take colorType, so we can stop calling SkColorTypeToBitmapConfig |
| 121 | // |
| 122 | bool SkImageDecoder::chooseFromOneChoice(SkColorType colorType, int width, int height) const { |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 123 | Chooser* chooser = fChooser; |
reed | 6c22573 | 2014-06-09 19:52:07 -0700 | [diff] [blame] | 124 | |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 125 | if (NULL == chooser) { // no chooser, we just say YES to decoding :) |
| 126 | return true; |
| 127 | } |
| 128 | chooser->begin(1); |
reed | 6c22573 | 2014-06-09 19:52:07 -0700 | [diff] [blame] | 129 | chooser->inspect(0, SkColorTypeToBitmapConfig(colorType), width, height); |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 130 | return chooser->choose() == 0; |
| 131 | } |
reed | 5926b86 | 2014-06-11 10:33:13 -0700 | [diff] [blame] | 132 | #endif |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 133 | |
| 134 | bool SkImageDecoder::allocPixelRef(SkBitmap* bitmap, |
| 135 | SkColorTable* ctable) const { |
reed | 8482504 | 2014-09-02 12:50:45 -0700 | [diff] [blame^] | 136 | return bitmap->tryAllocPixels(fAllocator, ctable); |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 137 | } |
| 138 | |
| 139 | /////////////////////////////////////////////////////////////////////////////// |
reed@android.com | b6137c3 | 2009-07-29 20:56:52 +0000 | [diff] [blame] | 140 | |
reed | 6c22573 | 2014-06-09 19:52:07 -0700 | [diff] [blame] | 141 | SkColorType SkImageDecoder::getPrefColorType(SrcDepth srcDepth, bool srcHasAlpha) const { |
reed | bfefc7c | 2014-06-12 17:40:00 -0700 | [diff] [blame] | 142 | SkColorType ct = fDefaultPref; |
reed | be08ace | 2014-07-08 11:15:59 -0700 | [diff] [blame] | 143 | if (fPreserveSrcDepth) { |
| 144 | switch (srcDepth) { |
| 145 | case kIndex_SrcDepth: |
| 146 | ct = kIndex_8_SkColorType; |
| 147 | break; |
| 148 | case k8BitGray_SrcDepth: |
| 149 | ct = kN32_SkColorType; |
| 150 | break; |
| 151 | case k32Bit_SrcDepth: |
| 152 | ct = kN32_SkColorType; |
| 153 | break; |
| 154 | } |
| 155 | } |
reed | bfefc7c | 2014-06-12 17:40:00 -0700 | [diff] [blame] | 156 | return ct; |
reed@android.com | 3f1f06a | 2010-03-03 21:04:12 +0000 | [diff] [blame] | 157 | } |
| 158 | |
reed | bfefc7c | 2014-06-12 17:40:00 -0700 | [diff] [blame] | 159 | bool SkImageDecoder::decode(SkStream* stream, SkBitmap* bm, SkColorType pref, Mode mode) { |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 160 | // we reset this to false before calling onDecode |
| 161 | fShouldCancelDecode = false; |
reed | bfefc7c | 2014-06-12 17:40:00 -0700 | [diff] [blame] | 162 | // assign this, for use by getPrefColorType(), in case fUsePrefTable is false |
reed@android.com | 3f1f06a | 2010-03-03 21:04:12 +0000 | [diff] [blame] | 163 | fDefaultPref = pref; |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 164 | |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 165 | // pass a temporary bitmap, so that if we return false, we are assured of |
| 166 | // leaving the caller's bitmap untouched. |
| 167 | SkBitmap tmp; |
reed@android.com | 3f1f06a | 2010-03-03 21:04:12 +0000 | [diff] [blame] | 168 | if (!this->onDecode(stream, &tmp, mode)) { |
reed@android.com | 62900b4 | 2009-02-11 15:07:19 +0000 | [diff] [blame] | 169 | return false; |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 170 | } |
reed@android.com | 62900b4 | 2009-02-11 15:07:19 +0000 | [diff] [blame] | 171 | bm->swap(tmp); |
| 172 | return true; |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 173 | } |
| 174 | |
reed | bfefc7c | 2014-06-12 17:40:00 -0700 | [diff] [blame] | 175 | bool SkImageDecoder::decodeSubset(SkBitmap* bm, const SkIRect& rect, SkColorType pref) { |
scroggo@google.com | 7e6fcee | 2013-05-03 20:14:28 +0000 | [diff] [blame] | 176 | // we reset this to false before calling onDecodeSubset |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 177 | fShouldCancelDecode = false; |
reed | bfefc7c | 2014-06-12 17:40:00 -0700 | [diff] [blame] | 178 | // assign this, for use by getPrefColorType(), in case fUsePrefTable is false |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 179 | fDefaultPref = pref; |
| 180 | |
scroggo@google.com | 7e6fcee | 2013-05-03 20:14:28 +0000 | [diff] [blame] | 181 | return this->onDecodeSubset(bm, rect); |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 182 | } |
| 183 | |
reed | bfefc7c | 2014-06-12 17:40:00 -0700 | [diff] [blame] | 184 | bool SkImageDecoder::buildTileIndex(SkStreamRewindable* stream, int *width, int *height) { |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 185 | // we reset this to false before calling onBuildTileIndex |
| 186 | fShouldCancelDecode = false; |
| 187 | |
| 188 | return this->onBuildTileIndex(stream, width, height); |
| 189 | } |
| 190 | |
scroggo@google.com | 7e6fcee | 2013-05-03 20:14:28 +0000 | [diff] [blame] | 191 | bool SkImageDecoder::cropBitmap(SkBitmap *dst, SkBitmap *src, int sampleSize, |
| 192 | int dstX, int dstY, int width, int height, |
| 193 | int srcX, int srcY) { |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 194 | int w = width / sampleSize; |
| 195 | int h = height / sampleSize; |
reed | 0689d7b | 2014-06-14 05:30:20 -0700 | [diff] [blame] | 196 | if (src->colorType() == kIndex_8_SkColorType) { |
scroggo@google.com | 7e6fcee | 2013-05-03 20:14:28 +0000 | [diff] [blame] | 197 | // kIndex8 does not allow drawing via an SkCanvas, as is done below. |
| 198 | // Instead, use extractSubset. Note that this shares the SkPixelRef and |
| 199 | // SkColorTable. |
| 200 | // FIXME: Since src is discarded in practice, this holds on to more |
| 201 | // pixels than is strictly necessary. Switch to a copy if memory |
| 202 | // savings are more important than speed here. This also means |
| 203 | // that the pixels in dst can not be reused (though there is no |
| 204 | // allocation, which was already done on src). |
| 205 | int x = (dstX - srcX) / sampleSize; |
| 206 | int y = (dstY - srcY) / sampleSize; |
| 207 | SkIRect subset = SkIRect::MakeXYWH(x, y, w, h); |
| 208 | return src->extractSubset(dst, subset); |
| 209 | } |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 210 | // if the destination has no pixels then we must allocate them. |
| 211 | if (dst->isNull()) { |
reed | 6c22573 | 2014-06-09 19:52:07 -0700 | [diff] [blame] | 212 | dst->setInfo(src->info().makeWH(w, h)); |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 213 | |
| 214 | if (!this->allocPixelRef(dst, NULL)) { |
| 215 | SkDEBUGF(("failed to allocate pixels needed to crop the bitmap")); |
scroggo@google.com | 7e6fcee | 2013-05-03 20:14:28 +0000 | [diff] [blame] | 216 | return false; |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 217 | } |
| 218 | } |
| 219 | // check to see if the destination is large enough to decode the desired |
| 220 | // region. If this assert fails we will just draw as much of the source |
| 221 | // into the destination that we can. |
scroggo@google.com | 7e6fcee | 2013-05-03 20:14:28 +0000 | [diff] [blame] | 222 | if (dst->width() < w || dst->height() < h) { |
| 223 | SkDEBUGF(("SkImageDecoder::cropBitmap does not have a large enough bitmap.\n")); |
| 224 | } |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 225 | |
| 226 | // Set the Src_Mode for the paint to prevent transparency issue in the |
| 227 | // dest in the event that the dest was being re-used. |
| 228 | SkPaint paint; |
| 229 | paint.setXfermodeMode(SkXfermode::kSrc_Mode); |
| 230 | |
| 231 | SkCanvas canvas(*dst); |
| 232 | canvas.drawSprite(*src, (srcX - dstX) / sampleSize, |
| 233 | (srcY - dstY) / sampleSize, |
| 234 | &paint); |
scroggo@google.com | 7e6fcee | 2013-05-03 20:14:28 +0000 | [diff] [blame] | 235 | return true; |
commit-bot@chromium.org | a936e37 | 2013-03-14 14:42:18 +0000 | [diff] [blame] | 236 | } |
| 237 | |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 238 | /////////////////////////////////////////////////////////////////////////////// |
| 239 | |
reed | bfefc7c | 2014-06-12 17:40:00 -0700 | [diff] [blame] | 240 | bool SkImageDecoder::DecodeFile(const char file[], SkBitmap* bm, SkColorType pref, Mode mode, |
| 241 | Format* format) { |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 242 | SkASSERT(file); |
| 243 | SkASSERT(bm); |
| 244 | |
scroggo@google.com | b5571b3 | 2013-09-25 21:34:24 +0000 | [diff] [blame] | 245 | SkAutoTUnref<SkStreamRewindable> stream(SkStream::NewFromFile(file)); |
mike@reedtribe.org | f381162 | 2013-03-19 02:18:33 +0000 | [diff] [blame] | 246 | if (stream.get()) { |
| 247 | if (SkImageDecoder::DecodeStream(stream, bm, pref, mode, format)) { |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 248 | bm->pixelRef()->setURI(file); |
tomhudson@google.com | 1a36621 | 2012-01-03 14:42:08 +0000 | [diff] [blame] | 249 | return true; |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 250 | } |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 251 | } |
| 252 | return false; |
| 253 | } |
| 254 | |
reed | bfefc7c | 2014-06-12 17:40:00 -0700 | [diff] [blame] | 255 | bool SkImageDecoder::DecodeMemory(const void* buffer, size_t size, SkBitmap* bm, SkColorType pref, |
| 256 | Mode mode, Format* format) { |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 257 | if (0 == size) { |
| 258 | return false; |
| 259 | } |
| 260 | SkASSERT(buffer); |
| 261 | |
| 262 | SkMemoryStream stream(buffer, size); |
reed@android.com | b3ade9d | 2009-06-15 13:04:45 +0000 | [diff] [blame] | 263 | return SkImageDecoder::DecodeStream(&stream, bm, pref, mode, format); |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 264 | } |
| 265 | |
reed | bfefc7c | 2014-06-12 17:40:00 -0700 | [diff] [blame] | 266 | bool SkImageDecoder::DecodeStream(SkStreamRewindable* stream, SkBitmap* bm, SkColorType pref, |
| 267 | Mode mode, Format* format) { |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 268 | SkASSERT(stream); |
| 269 | SkASSERT(bm); |
| 270 | |
| 271 | bool success = false; |
| 272 | SkImageDecoder* codec = SkImageDecoder::Factory(stream); |
| 273 | |
| 274 | if (NULL != codec) { |
| 275 | success = codec->decode(stream, bm, pref, mode); |
reed@android.com | b3ade9d | 2009-06-15 13:04:45 +0000 | [diff] [blame] | 276 | if (success && format) { |
| 277 | *format = codec->getFormat(); |
scroggo@google.com | 39edf4c | 2013-04-25 17:33:51 +0000 | [diff] [blame] | 278 | if (kUnknown_Format == *format) { |
| 279 | if (stream->rewind()) { |
| 280 | *format = GetStreamFormat(stream); |
| 281 | } |
| 282 | } |
reed@android.com | b3ade9d | 2009-06-15 13:04:45 +0000 | [diff] [blame] | 283 | } |
reed@android.com | 8a1c16f | 2008-12-17 15:59:43 +0000 | [diff] [blame] | 284 | delete codec; |
| 285 | } |
| 286 | return success; |
| 287 | } |