Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(581)

Unified Diff: gm/etc1bitmap.cpp

Issue 316813005: Add non power of two gm for ETC1 bitmap (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: More gyps Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | gyp/SampleApp.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gm/etc1bitmap.cpp
diff --git a/gm/etc1bitmap.cpp b/gm/etc1bitmap.cpp
index ec71d8daa56843344d4d95469f09ac018190e3fb..cdf8617a35627e317feb96de0ea8e09148265af0 100644
--- a/gm/etc1bitmap.cpp
+++ b/gm/etc1bitmap.cpp
@@ -12,6 +12,61 @@
#include "SkImageDecoder.h"
#include "SkOSFile.h"
+#ifndef SK_IGNORE_ETC1_SUPPORT
+
+#include "etc1.h"
+
+/**
+ * Remove the last row and column of ETC1 blocks, effectively
+ * making a texture that started as power of two into a texture
+ * that is no longer power of two...
+ */
+bool slice_etc1_data(void *data, int* width, int* height) {
+
+ // First, parse the data and get to it...
+ etc1_byte *origData = reinterpret_cast<etc1_byte *>(data);
+ if (!etc1_pkm_is_valid(origData)) {
+ return false;
+ }
+
+ int origW = etc1_pkm_get_width(origData);
+ int origH = etc1_pkm_get_height(origData);
+
+ int blockWidth = (origW + 3) >> 2;
+ int blockHeight = (origH + 3) >> 2;
+
+ // Make sure that we have blocks to trim off..
+ if (blockWidth < 2 || blockHeight < 2) {
+ return false;
+ }
+
+ int newWidth = (blockWidth - 1) << 2;
+ int newHeight = (blockHeight - 1) << 2;
+
+ size_t newDataSz = etc1_get_encoded_data_size(newWidth, newHeight) + ETC_PKM_HEADER_SIZE;
+ SkAutoMalloc am(newDataSz);
+
+ etc1_byte *newData = reinterpret_cast<etc1_byte *>(am.get());
+
+ etc1_pkm_format_header(newData, newWidth, newHeight);
+ newData += ETC_PKM_HEADER_SIZE;
+ origData += ETC_PKM_HEADER_SIZE;
+
+ for (int j = 0; j < blockHeight - 1; ++j) {
+ memcpy(newData, origData, (blockWidth - 1)*ETC1_ENCODED_BLOCK_SIZE);
+ origData += blockWidth*ETC1_ENCODED_BLOCK_SIZE;
+ newData += (blockWidth - 1)*ETC1_ENCODED_BLOCK_SIZE;
+ }
+
+ // Stick the data back whence it came
+ memcpy(data, am.get(), newDataSz);
+ *width = newWidth;
+ *height = newHeight;
+
+ return true;
+}
+#endif // SK_IGNORE_ETC1_SUPPORT
+
namespace skiagm {
/**
@@ -90,9 +145,76 @@ private:
typedef ETC1BitmapGM INHERITED;
};
+#ifndef SK_IGNORE_ETC1_SUPPORT
+/**
+ * Test decoding an image from a PKM file and then
+ * from non-power-of-two compressed ETC1 data. First slice
+ * off a row and column of blocks in order to make it non-power
+ * of two.
+ */
+class ETC1Bitmap_NPOT_GM : public GM {
+public:
+ ETC1Bitmap_NPOT_GM() { }
+ virtual ~ETC1Bitmap_NPOT_GM() { }
+
+protected:
+ virtual SkString onShortName() SK_OVERRIDE {
+ return SkString("etc1bitmap_npot");
+ }
+
+ virtual SkISize onISize() SK_OVERRIDE {
+ return make_isize(124, 124);
+ }
+
+ virtual void onDraw(SkCanvas* canvas) SK_OVERRIDE {
+
+ SkBitmap bm;
+ SkString filename = SkOSPath::SkPathJoin(
+ INHERITED::gResourcePath.c_str(), "mandrill_128.pkm");
+
+ SkAutoDataUnref fileData(SkData::NewFromFileName(filename.c_str()));
+ if (NULL == fileData) {
+ SkDebugf("Could not open the file. Did you forget to set the resourcePath?\n");
+ return;
+ }
+
+ SkAutoMalloc am(fileData->size());
+ memcpy(am.get(), fileData->data(), fileData->size());
+
+ int width, height;
+ if (!slice_etc1_data(am.get(), &width, &height)) {
+ SkDebugf("ETC1 Data is poorly formatted.\n");
+ return;
+ }
+
+ SkASSERT(124 == width);
+ SkASSERT(124 == height);
+
+ size_t dataSz = etc1_get_encoded_data_size(width, height) + ETC_PKM_HEADER_SIZE;
+ SkAutoDataUnref nonPOTData(SkData::NewWithCopy(am.get(), dataSz));
+
+ if (!SkInstallDiscardablePixelRef(
+ SkDecodingImageGenerator::Create(
+ nonPOTData, SkDecodingImageGenerator::Options()), &bm)) {
+ SkDebugf("Could not install discardable pixel ref.\n");
+ return;
+ }
+
+ canvas->drawBitmap(bm, 0, 0);
+ }
+
+private:
+ typedef GM INHERITED;
+};
+#endif // SK_IGNORE_ETC1_SUPPORT
+
} // namespace skiagm
//////////////////////////////////////////////////////////////////////////////
DEF_GM( return SkNEW(skiagm::ETC1Bitmap_PKM_GM); )
DEF_GM( return SkNEW(skiagm::ETC1Bitmap_KTX_GM); )
+
+#ifndef SK_IGNORE_ETC1_SUPPORT
+DEF_GM( return SkNEW(skiagm::ETC1Bitmap_NPOT_GM); )
+#endif // SK_IGNORE_ETC1_SUPPORT
« no previous file with comments | « no previous file | gyp/SampleApp.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698