From 7f8be9d89b7f294bf5e5d377908a5c74d2f4968f Mon Sep 17 00:00:00 2001 From: Andrew Flynn Date: Thu, 4 Jun 2015 17:11:06 -0400 Subject: [PATCH] notifyCarrierNetworkChange:TelephonyManager->CarrierService Per API review, move TelephonyManager.notifyCarrierNetworkChange() to CarrierService.notifyCarrierNetworkChange(). Underlying telephony implementation remains unchanged. Also minor tweaks to CarrierService: * Remove some unnecessary @hide * Remove final qualifier from onBind() so that subclasses can handle new internal callers that want to bind to it. Bug: 21572049 Bug: 21630803 Change-Id: Idcbcd107966fe92d7edc23c261e11db61160f615 --- api/current.txt | 4 +- api/system-current.txt | 4 +- .../service/carrier/CarrierService.java | 48 +++++++++++++++++-- .../android/telephony/TelephonyManager.java | 29 ----------- 4 files changed, 47 insertions(+), 38 deletions(-) diff --git a/api/current.txt b/api/current.txt index c9a8efb2360f7..ee1f14f5de883 100644 --- a/api/current.txt +++ b/api/current.txt @@ -28651,7 +28651,8 @@ package android.service.carrier { public abstract class CarrierService extends android.app.Service { ctor public CarrierService(); - method public final android.os.IBinder onBind(android.content.Intent); + method public final void notifyCarrierNetworkChange(boolean); + method public android.os.IBinder onBind(android.content.Intent); method public abstract android.os.PersistableBundle onLoadConfig(android.service.carrier.CarrierIdentifier); field public static final java.lang.String BIND_SERVICE_INTERFACE = "android.service.carrier.BindService"; field public static final java.lang.String CONFIG_SERVICE_INTERFACE = "android.service.carrier.ConfigService"; @@ -31215,7 +31216,6 @@ package android.telephony { method public boolean isVoiceCapable(); method public boolean isWorldPhone(); method public void listen(android.telephony.PhoneStateListener, int); - method public void notifyCarrierNetworkChange(boolean); method public java.lang.String sendEnvelopeWithStatus(java.lang.String); method public boolean setLine1NumberForDisplay(java.lang.String, java.lang.String); method public boolean setOperatorBrandOverride(java.lang.String); diff --git a/api/system-current.txt b/api/system-current.txt index 2f26891769dc4..aebc187aa1c95 100644 --- a/api/system-current.txt +++ b/api/system-current.txt @@ -30684,7 +30684,8 @@ package android.service.carrier { public abstract class CarrierService extends android.app.Service { ctor public CarrierService(); - method public final android.os.IBinder onBind(android.content.Intent); + method public final void notifyCarrierNetworkChange(boolean); + method public android.os.IBinder onBind(android.content.Intent); method public abstract android.os.PersistableBundle onLoadConfig(android.service.carrier.CarrierIdentifier); field public static final java.lang.String BIND_SERVICE_INTERFACE = "android.service.carrier.BindService"; field public static final java.lang.String CONFIG_SERVICE_INTERFACE = "android.service.carrier.ConfigService"; @@ -33459,7 +33460,6 @@ package android.telephony { method public boolean isWorldPhone(); method public void listen(android.telephony.PhoneStateListener, int); method public boolean needsOtaServiceProvisioning(); - method public void notifyCarrierNetworkChange(boolean); method public java.lang.String sendEnvelopeWithStatus(java.lang.String); method public void setDataEnabled(boolean); method public void setDataEnabled(int, boolean); diff --git a/core/java/android/service/carrier/CarrierService.java b/core/java/android/service/carrier/CarrierService.java index 4a4a37544a553..4d21939514b72 100644 --- a/core/java/android/service/carrier/CarrierService.java +++ b/core/java/android/service/carrier/CarrierService.java @@ -14,10 +14,15 @@ package android.service.carrier; +import android.annotation.CallSuper; import android.app.Service; import android.content.Intent; import android.os.IBinder; import android.os.PersistableBundle; +import android.os.RemoteException; +import android.os.ServiceManager; + +import com.android.internal.telephony.ITelephonyRegistry; /** * A service that exposes carrier-specific functionality to the system. @@ -45,10 +50,16 @@ public abstract class CarrierService extends Service { public static final String CONFIG_SERVICE_INTERFACE = "android.service.carrier.ConfigService"; public static final String BIND_SERVICE_INTERFACE = "android.service.carrier.BindService"; + private static ITelephonyRegistry sRegistry; + private final ICarrierService.Stub mStubWrapper; public CarrierService() { mStubWrapper = new ICarrierServiceWrapper(); + if (sRegistry == null) { + sRegistry = ITelephonyRegistry.Stub.asInterface( + ServiceManager.getService("telephony.registry")); + } } /** @@ -83,9 +94,39 @@ public abstract class CarrierService extends Service { */ public abstract PersistableBundle onLoadConfig(CarrierIdentifier id); - /** @hide */ + /** + * Informs the system of an intentional upcoming carrier network change by + * a carrier app. This call is optional and is only used to allow the + * system to provide alternative UI while telephony is performing an action + * that may result in intentional, temporary network lack of connectivity. + *

+ * Based on the active parameter passed in, this method will either show or + * hide the alternative UI. There is no timeout associated with showing + * this UX, so a carrier app must be sure to call with active set to false + * sometime after calling with it set to true. + *

+ * Requires Permission: + * {@link android.Manifest.permission#MODIFY_PHONE_STATE MODIFY_PHONE_STATE} + * Or the calling app has carrier privileges. + * @see {@link android.telephony.TelephonyManager#hasCarrierPrivileges} + * + * @param active Whether the carrier network change is or shortly will be + * active. Set this value to true to begin showing + * alternative UI and false to stop. + */ + public final void notifyCarrierNetworkChange(boolean active) { + try { + if (sRegistry != null) sRegistry.notifyCarrierNetworkChange(active); + } catch (RemoteException | NullPointerException ex) {} + } + + /** + * If overriding this method, call through to the super method for any unknown actions. + * {@inheritDoc} + */ @Override - public final IBinder onBind(Intent intent) { + @CallSuper + public IBinder onBind(Intent intent) { switch (intent.getAction()) { case CONFIG_SERVICE_INTERFACE: case BIND_SERVICE_INTERFACE: @@ -98,11 +139,8 @@ public abstract class CarrierService extends Service { /** * A wrapper around ICarrierService that forwards calls to implementations of * {@link CarrierService}. - * - * @hide */ private class ICarrierServiceWrapper extends ICarrierService.Stub { - @Override public PersistableBundle getCarrierConfig(CarrierIdentifier id) { return CarrierService.this.onLoadConfig(id); diff --git a/telephony/java/android/telephony/TelephonyManager.java b/telephony/java/android/telephony/TelephonyManager.java index f51cb65485f61..581a59ef16975 100644 --- a/telephony/java/android/telephony/TelephonyManager.java +++ b/telephony/java/android/telephony/TelephonyManager.java @@ -2059,35 +2059,6 @@ public class TelephonyManager { return false; } - /** - * Informs the system of an intentional upcoming carrier network change by - * a carrier app. This call is optional and is only used to allow the - * system to provide alternative UI while telephony is performing an action - * that may result in intentional, temporary network lack of connectivity. - *

- * Based on the active parameter passed in, this method will either show or - * hide the alternative UI. There is no timeout associated with showing - * this UX, so a carrier app must be sure to call with active set to false - * sometime after calling with it set to true. - *

- * Requires Permission: - * {@link android.Manifest.permission#MODIFY_PHONE_STATE MODIFY_PHONE_STATE} - * Or the calling app has carrier privileges. - * @see #hasCarrierPrivileges - * - * @param active Whether the carrier network change is or shortly will be - * active. Set this value to true to begin showing - * alternative UI and false to stop. - */ - public void notifyCarrierNetworkChange(boolean active) { - try { - if (sRegistry != null) - sRegistry.notifyCarrierNetworkChange(active); - } catch (RemoteException ex) { - } catch (NullPointerException ex) { - } - } - /** * Returns the alphabetic identifier associated with the line 1 number. * Return null if it is unavailable.