TextureView: destroy layer on destroyHardwareResources event

bug:30468770
Change-Id: I5174808e3fa6e4a87e3522246522c4e6d7ee2c3c
This commit is contained in:
sergeyv
2016-08-02 14:46:12 -07:00
parent 82c2e88890
commit 1c16c37d86

View File

@@ -218,35 +218,47 @@ public class TextureView extends View {
/** @hide */
@Override
protected void onDetachedFromWindowInternal() {
destroySurface();
destroyHardwareLayer();
releaseSurfaceTexture();
super.onDetachedFromWindowInternal();
}
private void destroySurface() {
/**
* @hide
*/
@Override
protected void destroyHardwareResources() {
destroyHardwareLayer();
mUpdateSurface = mSurface != null;
}
private void destroyHardwareLayer() {
if (mLayer != null) {
mLayer.detachSurfaceTexture();
boolean shouldRelease = true;
if (mListener != null) {
shouldRelease = mListener.onSurfaceTextureDestroyed(mSurface);
}
synchronized (mNativeWindowLock) {
nDestroyNativeWindow();
}
mLayer.destroy();
if (shouldRelease) mSurface.release();
mSurface = null;
mLayer = null;
// Make sure if/when new layer gets re-created, transform matrix will
// be re-applied.
mMatrixChanged = true;
mHadSurface = true;
}
}
private void releaseSurfaceTexture() {
boolean shouldRelease = true;
if (mListener != null) {
shouldRelease = mListener.onSurfaceTextureDestroyed(mSurface);
}
synchronized (mNativeWindowLock) {
nDestroyNativeWindow();
}
if (shouldRelease) {
mSurface.release();
}
mSurface = null;
mHadSurface = true;
}
/**
* The layer type of a TextureView is ignored since a TextureView is always
* considered to act as a hardware layer. The optional paint supplied to this
@@ -366,9 +378,9 @@ public class TextureView extends View {
// Create a new SurfaceTexture for the layer.
mSurface = new SurfaceTexture(false);
mLayer.setSurfaceTexture(mSurface);
nCreateNativeWindow(mSurface);
}
mSurface.setDefaultBufferSize(getWidth(), getHeight());
nCreateNativeWindow(mSurface);
mSurface.setOnFrameAvailableListener(mUpdateListener, mAttachInfo.mHandler);