commit 33c025cbfb642bd391db9ab83b84553286a0bf93
author John Reck <jreck@google.com> Mon Feb 29 15:49:41 2016 -0800
committer John Reck <jreck@google.com> Mon Feb 29 15:49:41 2016 -0800
tree a933e9ac39d686bf29469d0a17dc2eb91a37696d
parent 342006e91990f59b2e71c76a7d8589f7de571246

Only copyLayer if valid rendertarget

Bug: 27407637

Texture layers will set a GL_NONE rendertarget
until they have managed to get the real rendertarget
from SurfaceTexture. If, however, copyLayer is called
prior to any calls to updateTexImage() (for example, if
no frames have yet been produced), this will
result in trying to use a Texture with a GL_NONE rendertarget
which is bad.

Change-Id: Ib7285d799cb94ebd44f07a89f2ef2075d79ef71f

libs/hwui/LayerRenderer.cpp

diff --git a/libs/hwui/LayerRenderer.cpp b/libs/hwui/LayerRenderer.cpp
index e04b9a2..5bce8ac 100644
--- a/libs/hwui/LayerRenderer.cpp
+++ b/libs/hwui/LayerRenderer.cpp
@@ -353,7 +353,7 @@
 
 bool LayerRenderer::copyLayer(RenderState& renderState, Layer* layer, SkBitmap* bitmap) {
     Caches& caches = Caches::getInstance();
-    if (layer
+    if (layer && layer->getRenderTarget() != GL_NONE
             && bitmap->width() <= caches.maxTextureSize
             && bitmap->height() <= caches.maxTextureSize) {