Merge "Use dp units instead of px to draw profiling graph"

This commit is contained in:
Romain Guy
2013-01-07 10:59:53 -08:00
committed by Android (Google) Code Review

View File

@@ -29,6 +29,7 @@ import android.os.Looper;
import android.os.SystemClock; import android.os.SystemClock;
import android.os.SystemProperties; import android.os.SystemProperties;
import android.os.Trace; import android.os.Trace;
import android.util.DisplayMetrics;
import android.util.Log; import android.util.Log;
import com.google.android.gles_jni.EGLImpl; import com.google.android.gles_jni.EGLImpl;
@@ -1241,7 +1242,7 @@ public abstract class HardwareRenderer {
mFrameCount++; mFrameCount++;
debugDirtyRegions(dirty, canvas); debugDirtyRegions(dirty, canvas);
drawProfileData(); drawProfileData(attachInfo);
} }
onPostDraw(); onPostDraw();
@@ -1260,7 +1261,7 @@ public abstract class HardwareRenderer {
return false; return false;
} }
abstract void drawProfileData(); abstract void drawProfileData(View.AttachInfo attachInfo);
private Rect beginFrame(HardwareCanvas canvas, Rect dirty, int surfaceState) { private Rect beginFrame(HardwareCanvas canvas, Rect dirty, int surfaceState) {
// We had to change the current surface and/or context, redraw everything // We had to change the current surface and/or context, redraw everything
@@ -1465,15 +1466,17 @@ public abstract class HardwareRenderer {
static class Gl20Renderer extends GlRenderer { static class Gl20Renderer extends GlRenderer {
// TODO: Convert dimensions to dp instead of px // TODO: Convert dimensions to dp instead of px
private static final int PROFILE_DRAW_MARGIN = 0; private static final int PROFILE_DRAW_MARGIN = 0;
private static final int PROFILE_DRAW_WIDTH = 4; private static final int PROFILE_DRAW_WIDTH = 3;
private static final int[] PROFILE_DRAW_COLORS = { 0xcf3e66cc, 0xcfdc3912, 0xcfe69800 }; private static final int[] PROFILE_DRAW_COLORS = { 0xcf3e66cc, 0xcfdc3912, 0xcfe69800 };
private static final int PROFILE_DRAW_CURRENT_FRAME_COLOR = 0xcf5faa4d; private static final int PROFILE_DRAW_CURRENT_FRAME_COLOR = 0xcf5faa4d;
private static final int PROFILE_DRAW_THRESHOLD_COLOR = 0xff5faa4d; private static final int PROFILE_DRAW_THRESHOLD_COLOR = 0xff5faa4d;
private static final int PROFILE_DRAW_THRESHOLD_STROKE_WIDTH = 2; private static final int PROFILE_DRAW_THRESHOLD_STROKE_WIDTH = 2;
private static final int PROFILE_DRAW_PX_PER_MS = 10; private static final int PROFILE_DRAW_PX_PER_MS = 7;
private GLES20Canvas mGlCanvas; private GLES20Canvas mGlCanvas;
private DisplayMetrics mDisplayMetrics;
private static EGLSurface sPbuffer; private static EGLSurface sPbuffer;
private static final Object[] sPbufferLock = new Object[0]; private static final Object[] sPbufferLock = new Object[0];
@@ -1577,9 +1580,13 @@ public abstract class HardwareRenderer {
} }
@Override @Override
void drawProfileData() { void drawProfileData(View.AttachInfo attachInfo) {
if (mProfileEnabled && mProfileVisualizerEnabled) { if (mProfileEnabled && mProfileVisualizerEnabled) {
initProfileDrawData(); initProfileDrawData(attachInfo);
final int pxPerMs = (int) (PROFILE_DRAW_PX_PER_MS * mDisplayMetrics.density + 0.5f);
final int margin = (int) (PROFILE_DRAW_MARGIN * mDisplayMetrics.density + 0.5f);
final int width = (int) (PROFILE_DRAW_WIDTH * mDisplayMetrics.density + 0.5f);
int x = 0; int x = 0;
int count = 0; int count = 0;
@@ -1591,11 +1598,11 @@ public abstract class HardwareRenderer {
int index = count * 4; int index = count * 4;
if (i == mProfileCurrentFrame) current = index; if (i == mProfileCurrentFrame) current = index;
x += PROFILE_DRAW_MARGIN; x += margin;
int x2 = x + PROFILE_DRAW_WIDTH; int x2 = x + width;
int y2 = mHeight; int y2 = mHeight;
int y1 = (int) (y2 - mProfileData[i] * PROFILE_DRAW_PX_PER_MS); int y1 = (int) (y2 - mProfileData[i] * pxPerMs);
float[] r = mProfileRects[0]; float[] r = mProfileRects[0];
r[index] = x; r[index] = x;
@@ -1604,7 +1611,7 @@ public abstract class HardwareRenderer {
r[index + 3] = y2; r[index + 3] = y2;
y2 = y1; y2 = y1;
y1 = (int) (y2 - mProfileData[i + 1] * PROFILE_DRAW_PX_PER_MS); y1 = (int) (y2 - mProfileData[i + 1] * pxPerMs);
r = mProfileRects[1]; r = mProfileRects[1];
r[index] = x; r[index] = x;
@@ -1613,7 +1620,7 @@ public abstract class HardwareRenderer {
r[index + 3] = y2; r[index + 3] = y2;
y2 = y1; y2 = y1;
y1 = (int) (y2 - mProfileData[i + 2] * PROFILE_DRAW_PX_PER_MS); y1 = (int) (y2 - mProfileData[i + 2] * pxPerMs);
r = mProfileRects[2]; r = mProfileRects[2];
r[index] = x; r[index] = x;
@@ -1621,15 +1628,15 @@ public abstract class HardwareRenderer {
r[index + 2] = x2; r[index + 2] = x2;
r[index + 3] = y2; r[index + 3] = y2;
x += PROFILE_DRAW_WIDTH; x += width;
count++; count++;
} }
x += PROFILE_DRAW_MARGIN; x += margin;
drawGraph(count); drawGraph(count);
drawCurrentFrame(current); drawCurrentFrame(current);
drawThreshold(x); drawThreshold(x, pxPerMs);
} }
} }
@@ -1646,15 +1653,16 @@ public abstract class HardwareRenderer {
mProfileRects[2][index + 2], mProfileRects[0][index + 3], mProfilePaint); mProfileRects[2][index + 2], mProfileRects[0][index + 3], mProfilePaint);
} }
private void drawThreshold(int x) { private void drawThreshold(int x, int pxPerMs) {
mProfilePaint.setColor(PROFILE_DRAW_THRESHOLD_COLOR); mProfilePaint.setColor(PROFILE_DRAW_THRESHOLD_COLOR);
mProfilePaint.setStrokeWidth(PROFILE_DRAW_THRESHOLD_STROKE_WIDTH); mProfilePaint.setStrokeWidth((int)
int y = mHeight - 16 * 10; (PROFILE_DRAW_THRESHOLD_STROKE_WIDTH * mDisplayMetrics.density + 0.5f));
int y = mHeight - 16 * pxPerMs;
mGlCanvas.drawLine(0.0f, y, x, y, mProfilePaint); mGlCanvas.drawLine(0.0f, y, x, y, mProfilePaint);
mProfilePaint.setStrokeWidth(1.0f); mProfilePaint.setStrokeWidth(1.0f);
} }
private void initProfileDrawData() { private void initProfileDrawData(View.AttachInfo attachInfo) {
if (mProfileRects == null) { if (mProfileRects == null) {
mProfileRects = new float[PROFILE_FRAME_DATA_COUNT][]; mProfileRects = new float[PROFILE_FRAME_DATA_COUNT][];
for (int i = 0; i < mProfileRects.length; i++) { for (int i = 0; i < mProfileRects.length; i++) {
@@ -1663,6 +1671,11 @@ public abstract class HardwareRenderer {
} }
mProfilePaint = new Paint(); mProfilePaint = new Paint();
} }
if (mDisplayMetrics == null) {
mDisplayMetrics = new DisplayMetrics();
}
attachInfo.mDisplay.getMetrics(mDisplayMetrics);
} }
@Override @Override