Track input shaders java-side for heap tooling

Bug: 280338223
Test: runahat on test app
Change-Id: I5a853a8e6865d30e343a5f0cf57045e44798d9ec
This commit is contained in:
John Reck
2023-05-08 18:17:55 -04:00
parent ac175cc44f
commit 1a15acb484

View File

@@ -19,6 +19,7 @@ package android.graphics;
import android.annotation.ColorInt;
import android.annotation.ColorLong;
import android.annotation.NonNull;
import android.util.ArrayMap;
import android.view.Window;
import libcore.util.NativeAllocationRegistry;
@@ -255,6 +256,12 @@ public class RuntimeShader extends Shader {
*/
private long mNativeInstanceRuntimeShaderBuilder;
/**
* For tracking GC usage. Keep a java-side reference for reachable objects to
* enable better heap tracking & tooling support
*/
private ArrayMap<String, Shader> mShaderUniforms = new ArrayMap<>();
/**
* Creates a new RuntimeShader.
*
@@ -490,6 +497,7 @@ public class RuntimeShader extends Shader {
if (shader == null) {
throw new NullPointerException("The shader parameter must not be null");
}
mShaderUniforms.put(shaderName, shader);
nativeUpdateShader(
mNativeInstanceRuntimeShaderBuilder, shaderName, shader.getNativeInstance());
discardNativeInstance();
@@ -511,6 +519,7 @@ public class RuntimeShader extends Shader {
throw new NullPointerException("The shader parameter must not be null");
}
mShaderUniforms.put(shaderName, shader);
nativeUpdateShader(mNativeInstanceRuntimeShaderBuilder, shaderName,
shader.getNativeInstanceWithDirectSampling());
discardNativeInstance();