From 68221f5e05be846a1fee619ce6d0a50d7e625a4f Mon Sep 17 00:00:00 2001 From: Julia Reynolds Date: Tue, 14 Jun 2016 16:47:53 -0400 Subject: Fix accessibility for autorotate tile. The tile should be 'on' when rotation is not locked. Change-Id: I084be4ffb759bf20efe80358b5d103fec8892f76 Fixes: 29102740 --- .../src/com/android/systemui/qs/tiles/RotationLockTile.java | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/qs/tiles/RotationLockTile.java b/packages/SystemUI/src/com/android/systemui/qs/tiles/RotationLockTile.java index 521df37b732c..499eb50e9992 100644 --- a/packages/SystemUI/src/com/android/systemui/qs/tiles/RotationLockTile.java +++ b/packages/SystemUI/src/com/android/systemui/qs/tiles/RotationLockTile.java @@ -77,7 +77,7 @@ public class RotationLockTile extends QSTile { if (mController == null) return; MetricsLogger.action(mContext, getMetricsCategory(), !mState.value); final boolean newState = !mState.value; - mController.setRotationLocked(newState); + mController.setRotationLocked(!newState); refreshState(newState); } @@ -89,15 +89,10 @@ public class RotationLockTile extends QSTile { @Override protected void handleUpdateState(BooleanState state, Object arg) { if (mController == null) return; - final boolean rotationLocked = arg != null ? (Boolean) arg - : mController.isRotationLocked(); + final boolean rotationLocked = mController.isRotationLocked(); // TODO: Handle accessibility rotation lock and whatnot. -// state.visible = mController.isRotationLockAffordanceVisible(); - if (state.value == rotationLocked && state.contentDescription != null) { - // No change and initialized, no need to update all the values. - return; - } - state.value = rotationLocked; + + state.value = !rotationLocked; final boolean portrait = isCurrentOrientationLockPortrait(mController, mContext); if (rotationLocked) { final int label = portrait ? R.string.quick_settings_rotation_locked_portrait_label -- cgit v1.2.3-59-g8ed1b