am 4f7f6173: Make Keyboard.setShifted return void

* commit '4f7f61730cbd45871c1e9044da84b494831f97c3':
  Make Keyboard.setShifted return void
main
Tadashi G. Takaoka 2011-12-05 22:44:56 -08:00 committed by Android Git Automerger
commit c7e0281a09
5 changed files with 15 additions and 25 deletions

View File

@ -167,20 +167,14 @@ public class Keyboard {
return !mShiftLockKeys.isEmpty();
}
public boolean setShiftLocked(boolean newShiftLockState) {
public void setShiftLocked(boolean newShiftLockState) {
for (final Key key : mShiftLockKeys) {
// To represent "shift locked" state. The highlight is handled by background image that
// might be a StateListDrawable.
key.setHighlightOn(newShiftLockState);
// To represent "shifted" state. The key might have a shifted icon.
if (newShiftLockState && mShiftedIcons.containsKey(key)) {
key.setIcon(mShiftedIcons.get(key));
} else {
key.setIcon(mUnshiftedIcons.get(key));
}
key.setIcon(newShiftLockState ? mShiftedIcons.get(key) : mUnshiftedIcons.get(key));
}
mShiftState.setShiftLocked(newShiftLockState);
return true;
}
public boolean isShiftLocked() {
@ -191,15 +185,13 @@ public class Keyboard {
return mShiftState.isShiftLockShifted();
}
public boolean setShifted(boolean newShiftState) {
for (final Key key : mShiftKeys) {
if (!newShiftState && !mShiftState.isShiftLocked()) {
key.setIcon(mUnshiftedIcons.get(key));
} else if (newShiftState && !mShiftState.isShiftedOrShiftLocked()) {
key.setIcon(mShiftedIcons.get(key));
public void setShifted(boolean newShiftState) {
if (!mShiftState.isShiftLocked()) {
for (final Key key : mShiftKeys) {
key.setIcon(newShiftState ? mShiftedIcons.get(key) : mUnshiftedIcons.get(key));
}
}
return mShiftState.setShifted(newShiftState);
mShiftState.setShifted(newShiftState);
}
public boolean isShiftedOrShiftLocked() {

View File

@ -421,7 +421,6 @@ public class KeyboardSwitcher implements SharedPreferences.OnSharedPreferenceCha
return;
if (shiftMode == AUTOMATIC_SHIFT) {
latinKeyboard.setAutomaticTemporaryUpperCase();
mKeyboardView.invalidateAllKeys();
} else {
final boolean shifted = (shiftMode == MANUAL_SHIFT);
// On non-distinct multi touch panel device, we should also turn off the shift locked
@ -431,15 +430,15 @@ public class KeyboardSwitcher implements SharedPreferences.OnSharedPreferenceCha
if (!hasDistinctMultitouch() && !shifted && latinKeyboard.isShiftLocked()) {
latinKeyboard.setShiftLocked(false);
}
if (latinKeyboard.setShifted(shifted)) {
mKeyboardView.invalidateAllKeys();
}
latinKeyboard.setShifted(shifted);
}
mKeyboardView.invalidateAllKeys();
}
private void setShiftLocked(boolean shiftLocked) {
LatinKeyboard latinKeyboard = getLatinKeyboard();
if (latinKeyboard != null && latinKeyboard.setShiftLocked(shiftLocked)) {
if (latinKeyboard != null) {
latinKeyboard.setShiftLocked(shiftLocked);
mKeyboardView.invalidateAllKeys();
}
}

View File

@ -132,9 +132,8 @@ public class MiniKeyboardView extends KeyboardView implements MoreKeysPanel {
@Override
public void setShifted(boolean shifted) {
final Keyboard keyboard = getKeyboard();
if (keyboard.setShifted(shifted)) {
invalidateAllKeys();
}
keyboard.setShifted(shifted);
invalidateAllKeys();
}
@Override

View File

@ -24,6 +24,7 @@ public interface MoreKeysPanel extends PointerTracker.KeyEventHandler {
public boolean dismissMoreKeysPanel();
}
// TODO: Remove this method.
public void setShifted(boolean shifted);
/**

View File

@ -33,7 +33,7 @@ public class KeyboardShiftState {
private int mState = NORMAL;
public boolean setShifted(boolean newShiftState) {
public void setShifted(boolean newShiftState) {
final int oldState = mState;
if (newShiftState) {
switch (oldState) {
@ -61,7 +61,6 @@ public class KeyboardShiftState {
}
if (DEBUG)
Log.d(TAG, "setShifted(" + newShiftState + "): " + toString(oldState) + " > " + this);
return mState != oldState;
}
public void setShiftLocked(boolean newShiftLockState) {