Change encryption preference's summary accordingly
With this change a summary to the encryption preference was added.
415ce9a77d
The problem is that even if you're not encrypted, it says that your phone is.
Is very misleading to the user that don't bother to enter the actual preference
and want to see if they are. This fixes that.
This is cherry-picked from
https://android-review.googlesource.com/#/c/platform/packages/apps/Settings/+/517875/
Test: Go into Settings/Security & Location
Fixes: 68691666
Fixes: 68542516
Change-Id: Iaeaffd340f2e248748f244a947aa71eb8ad4ab94
This commit is contained in:
@@ -819,8 +819,10 @@
|
||||
<string name="encryption_and_credential_settings_title">Encryption & credentials</string>
|
||||
<!-- Security Settings screen Encryption and crendential summary -->
|
||||
<string name="encryption_and_credential_settings_summary" product="default">Phone encrypted</string>
|
||||
<string name="decryption_settings_summary" product="default">Phone not encrypted</string>
|
||||
<!-- Security Settings screen Encryption and crendential summary -->
|
||||
<string name="encryption_and_credential_settings_summary" product="tablet">Device encrypted</string>
|
||||
<string name="decryption_settings_summary" product="tablet">Device not encrypted</string>
|
||||
<!-- Security Settings screen setting option title for the item to take you to the lock screen preference screen [CHAR LIMIT=60] -->
|
||||
<string name="lockscreen_settings_title">Lock screen preferences</string>
|
||||
|
||||
|
@@ -56,7 +56,7 @@ public class EncryptionStatusPreferenceController extends BasePreferenceControll
|
||||
if (TextUtils.equals(getPreferenceKey(), PREF_KEY_ENCRYPTION_DETAIL_PAGE)) {
|
||||
preference.setFragment(CryptKeeperSettings.class.getName());
|
||||
}
|
||||
preference.setSummary(R.string.summary_placeholder);
|
||||
preference.setSummary(R.string.decryption_settings_summary);
|
||||
}
|
||||
|
||||
}
|
||||
|
@@ -84,7 +84,7 @@ public class EncryptionStatusPreferenceControllerTest {
|
||||
|
||||
mController.updateState(mPreference);
|
||||
|
||||
final CharSequence summary = mContext.getText(R.string.summary_placeholder);
|
||||
final CharSequence summary = mContext.getText(R.string.decryption_settings_summary);
|
||||
assertThat(mPreference.getSummary()).isEqualTo(summary);
|
||||
assertThat(mController.getPreferenceKey()).isNotEqualTo(PREF_KEY_ENCRYPTION_SECURITY_PAGE);
|
||||
assertThat(mPreference.getFragment()).isEqualTo(CryptKeeperSettings.class.getName());
|
||||
@@ -98,7 +98,7 @@ public class EncryptionStatusPreferenceControllerTest {
|
||||
|
||||
mController.updateState(mPreference);
|
||||
|
||||
final CharSequence summary = mContext.getText(R.string.summary_placeholder);
|
||||
final CharSequence summary = mContext.getText(R.string.decryption_settings_summary);
|
||||
assertThat(mPreference.getSummary()).isEqualTo(summary);
|
||||
|
||||
assertThat(mPreference.getFragment()).isNotEqualTo(CryptKeeperSettings.class.getName());
|
||||
|
Reference in New Issue
Block a user