Initial KTX encoder

src/images/SkImageDecoder_ktx.cpp

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkColorPriv.h"
 #include "SkImageDecoder.h"
+#include "SkPixelRef.h"
 #include "SkScaledBitmapSampler.h"
 #include "SkStream.h"
 #include "SkStreamHelpers.h"
 #include "SkTypes.h"
 
 #include "ktx.h"
 #include "etc1.h"
 
 /////////////////////////////////////////////////////////////////////////////////////////
 
@@ skipping 46 matching lines @@
         return false;
     }
 
     // Setup the sampler...
     SkScaledBitmapSampler sampler(width, height, this->getSampleSize());
 
     // Set the config...
     bm->setConfig(SkBitmap::kARGB_8888_Config,
                   sampler.scaledWidth(), sampler.scaledHeight(),
                   0,
-                  ktxFile.isRGBA8()? kUnpremul_SkAlphaType : kOpaque_SkAlphaType);
+                  ktxFile.isRGBA8()? kPremul_SkAlphaType : kOpaque_SkAlphaType);
     if (SkImageDecoder::kDecodeBounds_Mode == mode) {
         return true;
     }
     
     // If we've made it this far, then we know how to grok the data.
     if (!this->allocPixelRef(bm, NULL)) {
         return false;
     }
 
     // Lock the pixels, since we're about to write to them...
@@ skipping 41 matching lines @@
         srcRow += sampler.srcY0() * srcRowBytes;
         for (int y = 0; y < dstHeight; ++y) {
             sampler.next(srcRow);
             srcRow += sampler.srcDY() * srcRowBytes;
         }
 
         return true;
 
     } else if (ktxFile.isRGBA8()) {
 
+        // If we know that the image contains premultiplied alpha, then
+        // don't premultiply it upon decoding.
+        bool setRequireUnpremul = false;

[robertphillips, 2014/06/05 19:12:19]

Can we just do "SkString premulKey("KTXPremultipliedAlpha");" ?
Can we make it const?

[krajcevski, 2014/06/05 19:29:43]

Done.

+        SkString premulKey = SkString("KTXPremultipliedAlpha");
+        if (ktxFile.getValueForKey(premulKey) == SkString("True")) {
+            this->setRequireUnpremultipliedColors(true);
+            setRequireUnpremul = true;
+        }
+
         // Uncompressed RGBA data
         if (!sampler.begin(bm, SkScaledBitmapSampler::kRGBA, *this)) {
             return false;
         }
 
         // Just need to read RGBA pixels
         const int srcRowBytes = width * 4;
         const int dstHeight = sampler.scaledHeight();
         const uint8_t *srcRow = reinterpret_cast<const uint8_t *>(ktxFile.pixelData());
         srcRow += sampler.srcY0() * srcRowBytes;
         for (int y = 0; y < dstHeight; ++y) {
             sampler.next(srcRow);
             srcRow += sampler.srcDY() * srcRowBytes;
         }
 
+        // Reset this in case the decoder needs to be used again.
+        if (setRequireUnpremul) {
+            this->setRequireUnpremultipliedColors(false);
+        }
+
         return true;
     }
 
     return false;
 }
 
+///////////////////////////////////////////////////////////////////////////////
+

[robertphillips, 2014/06/05 19:12:19]

// ??

[krajcevski, 2014/06/05 19:29:43]

Done.

+class SkKTXImageEncoder : public SkImageEncoder {
+protected:
+    virtual bool onEncode(SkWStream* stream, const SkBitmap& bm, int quality) SK_OVERRIDE;
+
+private:
+    virtual bool encodePKM(SkWStream* stream, const SkData *data);
+    typedef SkImageEncoder INHERITED;
+};
+

[robertphillips, 2014/06/05 19:12:20]

Can we put onEncode before encodePKM in the file?

[krajcevski, 2014/06/05 19:29:43]

Done.

+bool SkKTXImageEncoder::encodePKM(SkWStream* stream, const SkData *data) {
+    const uint8_t* bytes = data->bytes();
+    SkASSERT(etc1_pkm_is_valid(bytes));
+
+    etc1_uint32 width = etc1_pkm_get_width(bytes);
+    etc1_uint32 height = etc1_pkm_get_height(bytes);
+
+    // ETC1 Data is stored as compressed 4x4 pixel blocks, so we must make sure
+    // that our dimensions are valid.
+    if (width == 0 || (width & 3) != 0 || height == 0 || (height & 3) != 0) {
+        return false;
+    }
+
+    // Advance pointer to etc1 data.
+    bytes += ETC_PKM_HEADER_SIZE;
+
+    return SkKTXFile::WriteETC1ToKTX(stream, bytes, width, height);
+}
+
+bool SkKTXImageEncoder::onEncode(SkWStream* stream, const SkBitmap& bitmap, int) {
+    SkAutoDataUnref data(bitmap.pixelRef()->refEncodedData());
+
+    // Is this even encoded data?
+    if (NULL != data) {
+        const uint8_t *bytes = data->bytes();
+        if (etc1_pkm_is_valid(bytes)) {

[robertphillips, 2014/06/05 19:12:20]

this-> ?

[krajcevski, 2014/06/05 19:29:43]

Done.

+            return encodePKM(stream, data);
+        }
+
+        // Is it a KTX file??
+        if (SkKTXFile::is_ktx(bytes)) {
+            return stream->write(bytes, data->size());
+        }
+        
+        // If it's neither a KTX nor a PKM, then we need to
+        // get at the actual pixels, so fall through and decompress...
+    }
+
+    return SkKTXFile::WriteBitmapToKTX(stream, bitmap);
+}
+
+
 /////////////////////////////////////////////////////////////////////////////////////////
 DEFINE_DECODER_CREATOR(KTXImageDecoder);
+DEFINE_ENCODER_CREATOR(KTXImageEncoder);
 /////////////////////////////////////////////////////////////////////////////////////////
 
 static SkImageDecoder* sk_libktx_dfactory(SkStreamRewindable* stream) {
     if (SkKTXFile::is_ktx(stream)) {
         return SkNEW(SkKTXImageDecoder);
     }
     return NULL;
 }
 
-static SkImageDecoder_DecodeReg gReg(sk_libktx_dfactory);
-
 static SkImageDecoder::Format get_format_ktx(SkStreamRewindable* stream) {
     if (SkKTXFile::is_ktx(stream)) {
         return SkImageDecoder::kKTX_Format;
     }
     return SkImageDecoder::kUnknown_Format;
 }
 
+SkImageEncoder* sk_libktx_efactory(SkImageEncoder::Type t) {
+    return (SkImageEncoder::kKTX_Type == t) ? SkNEW(SkKTXImageEncoder) : NULL;
+}
+
+static SkImageDecoder_DecodeReg gReg(sk_libktx_dfactory);
 static SkImageDecoder_FormatReg gFormatReg(get_format_ktx);
+static SkImageEncoder_EncodeReg gEReg(sk_libktx_efactory);