From c88e357d1ed9dadfc0efb3dfbe92f24460674ef5 Mon Sep 17 00:00:00 2001
From: Romain Guy <romainguy@google.com>
Date: Sat, 22 Jan 2011 00:32:12 -0800
Subject: Fix regression with hardware layers.

The renderer was checking whether the layer's region is a rect to
draw it with a simple quad. This test was done against the layer's
region structure, which was cleared after generating the mesh.
Unfortunately Region::isRect() returns true when the region is
empty, which was causing the renderer to always use a quad to
draw a mesh on screen.

Change-Id: If4b874579ee0cedcb67367c9efa5c0c98ca55231
---
 libs/hwui/OpenGLRenderer.cpp | 48 +++++++++++++++++++++++---------------------
 1 file changed, 25 insertions(+), 23 deletions(-)

(limited to 'libs/hwui/OpenGLRenderer.cpp')

diff --git a/libs/hwui/OpenGLRenderer.cpp b/libs/hwui/OpenGLRenderer.cpp
index b06bbd0aa543..92aa4bd7955c 100644
--- a/libs/hwui/OpenGLRenderer.cpp
+++ b/libs/hwui/OpenGLRenderer.cpp
@@ -1568,29 +1568,31 @@ void OpenGLRenderer::drawLayer(Layer* layer, float x, float y, SkPaint* paint) {
 
 
 #if RENDER_LAYERS_AS_REGIONS
-    if (layer->region.isRect()) {
-        const Rect r(x, y, x + layer->layer.getWidth(), y + layer->layer.getHeight());
-        composeLayerRect(layer, r);
-    } else if (!layer->region.isEmpty() && layer->mesh) {
-        const Rect& rect = layer->layer;
-
-        setupDraw();
-        setupDrawWithTexture();
-        setupDrawColor(alpha, alpha, alpha, alpha);
-        setupDrawColorFilter();
-        setupDrawBlending(layer->blend || layer->alpha < 255, layer->mode, false);
-        setupDrawProgram();
-        setupDrawDirtyRegionsDisabled();
-        setupDrawPureColorUniforms();
-        setupDrawColorFilterUniforms();
-        setupDrawTexture(layer->texture);
-        setupDrawModelViewTranslate(rect.left, rect.top, rect.right, rect.bottom);
-        setupDrawMesh(&layer->mesh[0].position[0], &layer->mesh[0].texture[0]);
-
-        glDrawElements(GL_TRIANGLES, layer->meshElementCount,
-                GL_UNSIGNED_SHORT, layer->meshIndices);
-
-        finishDrawTexture();
+    if (!layer->region.isEmpty()) {
+        if (layer->region.isRect()) {
+            const Rect r(x, y, x + layer->layer.getWidth(), y + layer->layer.getHeight());
+            composeLayerRect(layer, r);
+        } else if (layer->mesh) {
+            const Rect& rect = layer->layer;
+
+            setupDraw();
+            setupDrawWithTexture();
+            setupDrawColor(alpha, alpha, alpha, alpha);
+            setupDrawColorFilter();
+            setupDrawBlending(layer->blend || layer->alpha < 255, layer->mode, false);
+            setupDrawProgram();
+            setupDrawDirtyRegionsDisabled();
+            setupDrawPureColorUniforms();
+            setupDrawColorFilterUniforms();
+            setupDrawTexture(layer->texture);
+            setupDrawModelViewTranslate(rect.left, rect.top, rect.right, rect.bottom);
+            setupDrawMesh(&layer->mesh[0].position[0], &layer->mesh[0].texture[0]);
+
+            glDrawElements(GL_TRIANGLES, layer->meshElementCount,
+                    GL_UNSIGNED_SHORT, layer->meshIndices);
+
+            finishDrawTexture();
+        }
     }
 #else
     const Rect r(x, y, x + layer->layer.getWidth(), y + layer->layer.getHeight());
-- 
cgit v1.2.3-59-g8ed1b