Move EXTRA_SERVICE_STATE from Intent to ServiceState.

This extra is not exposed and is not used by anyone as such.
Leaving it there for internal use/unsupported app usage.

Test: basic sanity
Bug: 140908357
Merged-in: I9fe6e904291affb1cd7b705212d47525b61a5679
Change-Id: I9fe6e904291affb1cd7b705212d47525b61a5679
(cherry picked from commit bb61b17cd9)
This commit is contained in:
Amit Mahajan
2020-01-23 18:00:08 -08:00
parent f035fd91aa
commit 3ee618bbf5
2 changed files with 10 additions and 8 deletions

View File

@@ -4514,12 +4514,6 @@ public class Intent implements Parcelable, Cloneable {
@SystemApi
public static final String EXTRA_LTE_EARFCN_RSRP_BOOST = "LteEarfcnRsrpBoost";
/**
* An parcelable extra used with {@link #ACTION_SERVICE_STATE} representing the service state.
* @hide
*/
public static final String EXTRA_SERVICE_STATE = "android.intent.extra.SERVICE_STATE";
/**
* The name of the extra used to define the text to be processed, as a
* CharSequence. Note that this may be a styled CharSequence, so you must use

View File

@@ -317,6 +317,14 @@ public class ServiceState implements Parcelable {
*/
public static final int UNKNOWN_ID = -1;
/**
* A parcelable extra used with {@link Intent#ACTION_SERVICE_STATE} representing the service
* state.
* @hide
*/
private static final String EXTRA_SERVICE_STATE = "android.intent.extra.SERVICE_STATE";
private String mOperatorAlphaLong;
private String mOperatorAlphaShort;
private String mOperatorNumeric;
@@ -1292,7 +1300,7 @@ public class ServiceState implements Parcelable {
*/
@UnsupportedAppUsage
private void setFromNotifierBundle(Bundle m) {
ServiceState ssFromBundle = m.getParcelable(Intent.EXTRA_SERVICE_STATE);
ServiceState ssFromBundle = m.getParcelable(EXTRA_SERVICE_STATE);
if (ssFromBundle != null) {
copyFrom(ssFromBundle);
}
@@ -1310,7 +1318,7 @@ public class ServiceState implements Parcelable {
*/
@SystemApi
public void fillInNotifierBundle(@NonNull Bundle m) {
m.putParcelable(Intent.EXTRA_SERVICE_STATE, this);
m.putParcelable(EXTRA_SERVICE_STATE, this);
// serviceState already consists of below entries.
// for backward compatibility, we continue fill in below entries.
m.putInt("voiceRegState", mVoiceRegState);