nyancrimew
/
LatinIME
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

am 877157e1: Optimize keyboard drawing code a bit 

* commit '877157e1f5ed44d6490eaf8fab430f9a73caf34d':
  Optimize keyboard drawing code a bit
main
Tadashi G. Takaoka 2012-08-09 00:59:26 -07:00 committed by Android Git Automerger
parent 80d63d2263 877157e1f5
commit 03c580e801
1 changed files with 10 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
java/src/com/android/inputmethod/keyboard/KeyboardView.java
View File

@ -111,8 +111,6 @@ public class KeyboardView extends View implements PointerTracker.DrawingProxy {
// Drawing // Drawing
/** True if the entire keyboard needs to be dimmed. */ /** True if the entire keyboard needs to be dimmed. */
private boolean mNeedsToDimEntireKeyboard; private boolean mNeedsToDimEntireKeyboard;
/** Whether the keyboard bitmap buffer needs to be redrawn before it's blitted. **/
private boolean mBufferNeedsUpdate;
/** True if all keys should be drawn */ /** True if all keys should be drawn */
private boolean mInvalidateAllKeys; private boolean mInvalidateAllKeys;
/** The keys that should be drawn */ /** The keys that should be drawn */
@ -459,8 +457,9 @@ public class KeyboardView extends View implements PointerTracker.DrawingProxy {
onDrawKeyboard(canvas); onDrawKeyboard(canvas);
return; return;
} }
if (mBufferNeedsUpdate || mOffscreenBuffer == null) {
mBufferNeedsUpdate = false; final boolean bufferNeedsUpdates = mInvalidateAllKeys || !mInvalidatedKeys.isEmpty();
if (bufferNeedsUpdates || mOffscreenBuffer == null) {
if (maybeAllocateOffscreenBuffer()) { if (maybeAllocateOffscreenBuffer()) {
mInvalidateAllKeys = true; mInvalidateAllKeys = true;
// TODO: Stop using the offscreen canvas even when in software rendering // TODO: Stop using the offscreen canvas even when in software rendering
@ -524,14 +523,13 @@ public class KeyboardView extends View implements PointerTracker.DrawingProxy {
} }
if (!isHardwareAccelerated) { if (!isHardwareAccelerated) {
canvas.clipRegion(mClipRegion, Region.Op.REPLACE); canvas.clipRegion(mClipRegion, Region.Op.REPLACE);
}
// Draw keyboard background. // Draw keyboard background.
canvas.drawColor(Color.BLACK, PorterDuff.Mode.CLEAR); canvas.drawColor(Color.BLACK, PorterDuff.Mode.CLEAR);
final Drawable background = getBackground(); final Drawable background = getBackground();
if (background != null) { if (background != null) {
background.draw(canvas); background.draw(canvas);
} }
}
// TODO: Confirm if it's really required to draw all keys when hardware acceleration is on. // TODO: Confirm if it's really required to draw all keys when hardware acceleration is on.
if (drawAllKeys || isHardwareAccelerated) { if (drawAllKeys || isHardwareAccelerated) {
@ -1048,7 +1046,6 @@ public class KeyboardView extends View implements PointerTracker.DrawingProxy {
public void invalidateAllKeys() { public void invalidateAllKeys() {
mInvalidatedKeys.clear(); mInvalidatedKeys.clear();
mInvalidateAllKeys = true; mInvalidateAllKeys = true;
mBufferNeedsUpdate = true;
invalidate(); invalidate();
} }
@ -1066,9 +1063,7 @@ public class KeyboardView extends View implements PointerTracker.DrawingProxy {
mInvalidatedKeys.add(key); mInvalidatedKeys.add(key);
final int x = key.mX + getPaddingLeft(); final int x = key.mX + getPaddingLeft();
final int y = key.mY + getPaddingTop(); final int y = key.mY + getPaddingTop();
mWorkingRect.set(x, y, x + key.mWidth, y + key.mHeight); invalidate(x, y, x + key.mWidth, y + key.mHeight);
mBufferNeedsUpdate = true;
invalidate(mWorkingRect);
} }
public void closing() { public void closing() {