commit 4bb942083a0d4db746adf95349108dd8ef842e32
author Romain Guy <romainguy@google.com> Tue Oct 12 15:59:26 2010 -0700
committer Romain Guy <romainguy@google.com> Tue Oct 12 15:59:26 2010 -0700
tree 7707bc9551f3db4ad0561c30c73293e4a0edfbf2
parent 5a3b8e698e276839e502dcf41894b56ad59695c3

Optimize 9patch rendering.

This change detects empty quads in 9patches and removes them from
the mesh to avoid unnecessary blending.

Change-Id: I4500566fb4cb6845d64dcb59b522c0be7a0ec704

libs/hwui/PatchCache.cpp

diff --git a/libs/hwui/PatchCache.cpp b/libs/hwui/PatchCache.cpp
index eca5e4d..4762d21 100644
--- a/libs/hwui/PatchCache.cpp
+++ b/libs/hwui/PatchCache.cpp
@@ -17,7 +17,6 @@
 #define LOG_TAG "OpenGLRenderer"
 
 #include <utils/Log.h>
-#include <utils/ResourceTypes.h>
 
 #include "PatchCache.h"
 #include "Properties.h"
@@ -45,7 +44,7 @@
 
 void PatchCache::clear() {
     size_t count = mCache.size();
-    for (int i = 0; i < count; i++) {
+    for (size_t i = 0; i < count; i++) {
         delete mCache.valueAt(i);
     }
     mCache.clear();
@@ -53,11 +52,23 @@
 
 Patch* PatchCache::get(const float bitmapWidth, const float bitmapHeight,
         const float pixelWidth, const float pixelHeight,
-        const int32_t* xDivs, const int32_t* yDivs,
-        const uint32_t width, const uint32_t height) {
+        const int32_t* xDivs, const int32_t* yDivs, const uint32_t* colors,
+        const uint32_t width, const uint32_t height, const int8_t numColors) {
+
+    int8_t transparentQuads = 0;
+    uint32_t colorKey = 0;
+
+    if (uint8_t(numColors) < sizeof(uint32_t) * 4) {
+        for (int8_t i = 0; i < numColors; i++) {
+            if (colors[i] == 0x0) {
+                transparentQuads++;
+                colorKey |= 0x1 << i;
+            }
+        }
+    }
 
     const PatchDescription description(bitmapWidth, bitmapHeight,
-            pixelWidth, pixelHeight, width, height);
+            pixelWidth, pixelHeight, width, height, transparentQuads, colorKey);
 
     ssize_t index = mCache.indexOfKey(description);
     Patch* mesh = NULL;
@@ -66,11 +77,13 @@
     }
 
     if (!mesh) {
-        PATCH_LOGD("Creating new patch mesh, w=%d h=%d", width, height);
+        PATCH_LOGD("New patch mesh "
+                "xCount=%d yCount=%d, w=%.2f h=%.2f, bw=%.2f bh=%.2f",
+                width, height, pixelWidth, pixelHeight, bitmapWidth, bitmapHeight);
 
-        mesh = new Patch(width, height);
+        mesh = new Patch(width, height, transparentQuads);
         mesh->updateVertices(bitmapWidth, bitmapHeight, 0.0f, 0.0f,
-                pixelWidth, pixelHeight, xDivs, yDivs, width, height);
+                pixelWidth, pixelHeight, xDivs, yDivs, width, height, colorKey);
 
         if (mCache.size() >= mMaxEntries) {
             delete mCache.valueAt(mCache.size() - 1);