am 82b4374c: Merge "[Rlog48b] Better visual indicator for logging state"

# Via Android (Google) Code Review (1) and Kurt Partridge (1)
* commit '82b4374c75ba64dd736b9509f27389a38b92c8ed':
  [Rlog48b] Better visual indicator for logging state
main
Kurt Partridge 2013-02-04 10:07:50 -08:00 committed by Android Git Automerger
commit 32d171225b
2 changed files with 23 additions and 3 deletions

View File

@ -130,4 +130,8 @@ public class Replayer {
handler.postAtTime(callback, presentDoneTime + 1);
}
}
public boolean isReplaying() {
return mIsReplaying;
}
}

View File

@ -819,7 +819,8 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang
}
private boolean isAllowedToLog() {
return !mIsPasswordView && !mIsLoggingSuspended && sIsLogging && !mInFeedbackDialog;
return !mIsPasswordView && !mIsLoggingSuspended && sIsLogging && !mInFeedbackDialog
&& !isReplaying();
}
public void requestIndicatorRedraw() {
@ -832,15 +833,30 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang
mMainKeyboardView.invalidateAllKeys();
}
private boolean isReplaying() {
return mReplayer.isReplaying();
}
private int getIndicatorColor() {
if (isMakingUserRecording()) {
return Color.YELLOW;
}
if (isReplaying()) {
return Color.GREEN;
}
return Color.RED;
}
public void paintIndicator(KeyboardView view, Paint paint, Canvas canvas, int width,
int height) {
// TODO: Reimplement using a keyboard background image specific to the ResearchLogger
// and remove this method.
// The check for MainKeyboardView ensures that the indicator only decorates the main
// keyboard, not every keyboard.
if (IS_SHOWING_INDICATOR && isAllowedToLog() && view instanceof MainKeyboardView) {
if (IS_SHOWING_INDICATOR && (isAllowedToLog() || isReplaying())
&& view instanceof MainKeyboardView) {
final int savedColor = paint.getColor();
paint.setColor(isMakingUserRecording() ? Color.YELLOW : Color.RED);
paint.setColor(getIndicatorColor());
final Style savedStyle = paint.getStyle();
paint.setStyle(Style.STROKE);
final float savedStrokeWidth = paint.getStrokeWidth();