Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Dejank: don't allocate when scrolling lists

Browse Source 
The new display list properties introduces in JB were causing numerous
and expensive memory allocations while scrolling lists. During a scroll
ListView sometimes attempts to apply an offset to views before they are
drawn for the first time. This had the side effect of generating a new
IllegalStateException and its entire stack trace. The exception was
caught inside the display list and never seen by users.

Generating an exception is very expensive both in terms of allocated
memory and CPU time spent crawling the stack.

List scrolls/flings are a common case of this issue but it also happens
during various types of animations. A simple alpha animation, for instance,
can cause the problem to occur.

Another side effect of this issue is more frequent and longer GC pauses.

Change-Id: Ic1b37cc84f7c8f290209cfb990d030e96d6e0dc7
This commit is contained in:
Romain Guy
2012-06-20 23:59:06 -07:00
parent d7c00d2bdd
commit 0d6f4c06df
1 changed files with 54 additions and 100 deletions
Download Patch File Download Diff File Expand all files Collapse all files

154
core/java/android/view/GLES20DisplayList.java
View File

@@ -43,6 +43,10 @@ class GLES20DisplayList extends DisplayList {
mName = name;
}
boolean hasNativeDisplayList() {
return mValid && mFinalizer != null;
}
int getNativeDisplayList() {
if (!mValid || mFinalizer == null) {
throw new IllegalStateException("The display list is not valid.");
@@ -110,229 +114,179 @@ class GLES20DisplayList extends DisplayList {
@Override
public void setCaching(boolean caching) {
try {
nSetCaching(getNativeDisplayList(), caching);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetCaching(mFinalizer.mNativeDisplayList, caching);
}
}
@Override
public void setClipChildren(boolean clipChildren) {
try {
nSetClipChildren(getNativeDisplayList(), clipChildren);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetClipChildren(mFinalizer.mNativeDisplayList, clipChildren);
}
}
@Override
public void setStaticMatrix(Matrix matrix) {
try {
nSetStaticMatrix(getNativeDisplayList(), matrix.native_instance);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetStaticMatrix(mFinalizer.mNativeDisplayList, matrix.native_instance);
}
}
@Override
public void setAnimationMatrix(Matrix matrix) {
try {
nSetAnimationMatrix(getNativeDisplayList(),
if (hasNativeDisplayList()) {
nSetAnimationMatrix(mFinalizer.mNativeDisplayList,
(matrix != null) ? matrix.native_instance : 0);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
}
}
@Override
public void setAlpha(float alpha) {
try {
nSetAlpha(getNativeDisplayList(), alpha);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetAlpha(mFinalizer.mNativeDisplayList, alpha);
}
}
@Override
public void setHasOverlappingRendering(boolean hasOverlappingRendering) {
try {
nSetHasOverlappingRendering(getNativeDisplayList(), hasOverlappingRendering);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetHasOverlappingRendering(mFinalizer.mNativeDisplayList, hasOverlappingRendering);
}
}
@Override
public void setTranslationX(float translationX) {
try {
nSetTranslationX(getNativeDisplayList(), translationX);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetTranslationX(mFinalizer.mNativeDisplayList, translationX);
}
}
@Override
public void setTranslationY(float translationY) {
try {
nSetTranslationY(getNativeDisplayList(), translationY);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetTranslationY(mFinalizer.mNativeDisplayList, translationY);
}
}
@Override
public void setRotation(float rotation) {
try {
nSetRotation(getNativeDisplayList(), rotation);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetRotation(mFinalizer.mNativeDisplayList, rotation);
}
}
@Override
public void setRotationX(float rotationX) {
try {
nSetRotationX(getNativeDisplayList(), rotationX);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetRotationX(mFinalizer.mNativeDisplayList, rotationX);
}
}
@Override
public void setRotationY(float rotationY) {
try {
nSetRotationY(getNativeDisplayList(), rotationY);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetRotationY(mFinalizer.mNativeDisplayList, rotationY);
}
}
@Override
public void setScaleX(float scaleX) {
try {
nSetScaleX(getNativeDisplayList(), scaleX);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetScaleX(mFinalizer.mNativeDisplayList, scaleX);
}
}
@Override
public void setScaleY(float scaleY) {
try {
nSetScaleY(getNativeDisplayList(), scaleY);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetScaleY(mFinalizer.mNativeDisplayList, scaleY);
}
}
@Override
public void setTransformationInfo(float alpha, float translationX, float translationY,
float rotation, float rotationX, float rotationY, float scaleX, float scaleY) {
try {
nSetTransformationInfo(getNativeDisplayList(), alpha, translationX, translationY,
if (hasNativeDisplayList()) {
nSetTransformationInfo(mFinalizer.mNativeDisplayList, alpha, translationX, translationY,
rotation, rotationX, rotationY, scaleX, scaleY);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
}
}
@Override
public void setPivotX(float pivotX) {
try {
nSetPivotX(getNativeDisplayList(), pivotX);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetPivotX(mFinalizer.mNativeDisplayList, pivotX);
}
}
@Override
public void setPivotY(float pivotY) {
try {
nSetPivotY(getNativeDisplayList(), pivotY);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetPivotY(mFinalizer.mNativeDisplayList, pivotY);
}
}
@Override
public void setCameraDistance(float distance) {
try {
nSetCameraDistance(getNativeDisplayList(), distance);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetCameraDistance(mFinalizer.mNativeDisplayList, distance);
}
}
@Override
public void setLeft(int left) {
try {
nSetLeft(getNativeDisplayList(), left);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetLeft(mFinalizer.mNativeDisplayList, left);
}
}
@Override
public void setTop(int top) {
try {
nSetTop(getNativeDisplayList(), top);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetTop(mFinalizer.mNativeDisplayList, top);
}
}
@Override
public void setRight(int right) {
try {
nSetRight(getNativeDisplayList(), right);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetRight(mFinalizer.mNativeDisplayList, right);
}
}
@Override
public void setBottom(int bottom) {
try {
nSetBottom(getNativeDisplayList(), bottom);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetBottom(mFinalizer.mNativeDisplayList, bottom);
}
}
@Override
public void setLeftTop(int left, int top) {
try {
nSetLeftTop(getNativeDisplayList(), left, top);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetLeftTop(mFinalizer.mNativeDisplayList, left, top);
}
}
@Override
public void setLeftTopRightBottom(int left, int top, int right, int bottom) {
try {
nSetLeftTopRightBottom(getNativeDisplayList(), left, top, right, bottom);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nSetLeftTopRightBottom(mFinalizer.mNativeDisplayList, left, top, right, bottom);
}
}
@Override
public void offsetLeftRight(int offset) {
try {
nOffsetLeftRight(getNativeDisplayList(), offset);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nOffsetLeftRight(mFinalizer.mNativeDisplayList, offset);
}
}
@Override
public void offsetTopBottom(int offset) {
try {
nOffsetTopBottom(getNativeDisplayList(), offset);
} catch (IllegalStateException e) {
// invalid DisplayList okay: we'll set current values the next time we render to it
if (hasNativeDisplayList()) {
nOffsetTopBottom(mFinalizer.mNativeDisplayList, offset);
}
}