From 5cd5c3f7e4df99f47e3def6e3707b9eb8205aef3 Mon Sep 17 00:00:00 2001
From: Romain Guy <romainguy@google.com>
Date: Mon, 17 Oct 2011 17:10:02 -0700
Subject: [PATCH] Return early when we cannot allocate a hardware layer Bug
 #5462308

Change-Id: I52dab809662f6f42a49ca03edc50f6b98fc35a06
---
 core/java/android/view/View.java | 12 ++++++++++++
 libs/hwui/LayerRenderer.cpp      |  3 ++-
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/core/java/android/view/View.java b/core/java/android/view/View.java
index 86be28a534da1..4385d5a2c0c25 100644
--- a/core/java/android/view/View.java
+++ b/core/java/android/view/View.java
@@ -10092,8 +10092,20 @@ public class View implements Drawable.Callback, Drawable.Callback2, KeyEvent.Cal
                 mLocalDirtyRect.setEmpty();
             }
 
+            // The layer is not valid if the underlying GPU resources cannot be allocated
+            if (!mHardwareLayer.isValid()) {
+                return null;
+            }
+
             HardwareCanvas currentCanvas = mAttachInfo.mHardwareCanvas;
             final HardwareCanvas canvas = mHardwareLayer.start(currentCanvas);
+
+            // Make sure all the GPU resources have been properly allocated
+            if (canvas == null) {
+                mHardwareLayer.end(currentCanvas);
+                return null;
+            }
+
             mAttachInfo.mHardwareCanvas = canvas;
             try {
                 canvas.setViewport(width, height);
diff --git a/libs/hwui/LayerRenderer.cpp b/libs/hwui/LayerRenderer.cpp
index dfcc5ea4cf0f9..e38b4794bcae2 100644
--- a/libs/hwui/LayerRenderer.cpp
+++ b/libs/hwui/LayerRenderer.cpp
@@ -216,7 +216,8 @@ Layer* LayerRenderer::createLayer(uint32_t width, uint32_t height, bool isOpaque
         layer->allocateTexture(GL_RGBA, GL_UNSIGNED_BYTE);
 
         if (glGetError() != GL_NO_ERROR) {
-            LOGD("Could not allocate texture");
+            LOGD("Could not allocate texture for layer (fbo=%d %dx%d)",
+                    fbo, width, height);
 
             glBindFramebuffer(GL_FRAMEBUFFER, previousFbo);
             Caches::getInstance().fboCache.put(fbo);