Rework platform version to hide codenames.
The public platform version no longer can be a codename, it is always the most recently released platform. A new build property and API provides either the offical version or the current codename as appropriate. This will avoid breaking apps that look at the platform version while development is under a codename. Bug: 143175463 Test: manual Change-Id: I711ca20c4c8ce389697e940696051a336f5fd808
This commit is contained in:
@@ -737,7 +737,7 @@ public class PrivateVolumeSettings extends SettingsPreferenceFragment {
|
|||||||
public Dialog onCreateDialog(Bundle savedInstanceState) {
|
public Dialog onCreateDialog(Bundle savedInstanceState) {
|
||||||
return new AlertDialog.Builder(getActivity())
|
return new AlertDialog.Builder(getActivity())
|
||||||
.setMessage(getContext().getString(R.string.storage_detail_dialog_system,
|
.setMessage(getContext().getString(R.string.storage_detail_dialog_system,
|
||||||
Build.VERSION.RELEASE))
|
Build.VERSION.RELEASE_OR_CODENAME))
|
||||||
.setPositiveButton(android.R.string.ok, null)
|
.setPositiveButton(android.R.string.ok, null)
|
||||||
.create();
|
.create();
|
||||||
}
|
}
|
||||||
|
@@ -70,7 +70,7 @@ public class FirmwareVersionDetailPreferenceController extends BasePreferenceCon
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
public CharSequence getSummary() {
|
public CharSequence getSummary() {
|
||||||
return Build.VERSION.RELEASE;
|
return Build.VERSION.RELEASE_OR_CODENAME;
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
@@ -34,6 +34,6 @@ public class FirmwareVersionPreferenceController extends BasePreferenceControlle
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
public CharSequence getSummary() {
|
public CharSequence getSummary() {
|
||||||
return Build.VERSION.RELEASE;
|
return Build.VERSION.RELEASE_OR_CODENAME;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@@ -89,7 +89,7 @@ public class SystemUpdatePreferenceController extends BasePreferenceController {
|
|||||||
@Override
|
@Override
|
||||||
public CharSequence getSummary() {
|
public CharSequence getSummary() {
|
||||||
CharSequence summary = mContext.getString(R.string.android_version_summary,
|
CharSequence summary = mContext.getString(R.string.android_version_summary,
|
||||||
Build.VERSION.RELEASE);
|
Build.VERSION.RELEASE_OR_CODENAME);
|
||||||
final FutureTask<Bundle> bundleFutureTask = new FutureTask<>(
|
final FutureTask<Bundle> bundleFutureTask = new FutureTask<>(
|
||||||
// Put the API call in a future to avoid StrictMode violation.
|
// Put the API call in a future to avoid StrictMode violation.
|
||||||
() -> mUpdateManager.retrieveSystemUpdateInfo());
|
() -> mUpdateManager.retrieveSystemUpdateInfo());
|
||||||
|
@@ -64,7 +64,7 @@ public class FirmwareVersionDetailPreferenceControllerTest {
|
|||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void getSummary_shouldGetBuildVersion() {
|
public void getSummary_shouldGetBuildVersion() {
|
||||||
assertThat(mController.getSummary()).isEqualTo(Build.VERSION.RELEASE);
|
assertThat(mController.getSummary()).isEqualTo(Build.VERSION.RELEASE_OR_CODENAME);
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
|
@@ -65,6 +65,6 @@ public class FirmwareVersionPreferenceControllerTest {
|
|||||||
public void updatePreference_shouldSetSummaryToBuildNumber() {
|
public void updatePreference_shouldSetSummaryToBuildNumber() {
|
||||||
mController.updateState(mPreference);
|
mController.updateState(mPreference);
|
||||||
|
|
||||||
assertThat(mPreference.getSummary()).isEqualTo(Build.VERSION.RELEASE);
|
assertThat(mPreference.getSummary()).isEqualTo(Build.VERSION.RELEASE_OR_CODENAME);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@@ -139,7 +139,7 @@ public class SystemUpdatePreferenceControllerTest {
|
|||||||
|
|
||||||
assertThat(mPreference.getSummary())
|
assertThat(mPreference.getSummary())
|
||||||
.isEqualTo(mContext.getString(R.string.android_version_summary,
|
.isEqualTo(mContext.getString(R.string.android_version_summary,
|
||||||
Build.VERSION.RELEASE));
|
Build.VERSION.RELEASE_OR_CODENAME));
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
|
Reference in New Issue
Block a user