diff --git a/telephony/java/com/android/internal/telephony/DataConnectionTracker.java b/telephony/java/com/android/internal/telephony/DataConnectionTracker.java index 5d6aaa6b2f44e..2c26f62b9a166 100644 --- a/telephony/java/com/android/internal/telephony/DataConnectionTracker.java +++ b/telephony/java/com/android/internal/telephony/DataConnectionTracker.java @@ -509,7 +509,6 @@ public abstract class DataConnectionTracker extends Handler { protected abstract void onVoiceCallEnded(); protected abstract void onCleanUpConnection(boolean tearDown, int apnId, String reason); protected abstract void onCleanUpAllConnections(String cause); - protected abstract boolean isDataPossible(); protected abstract boolean isDataPossible(String apnType); @Override @@ -752,7 +751,7 @@ public abstract class DataConnectionTracker extends Handler { protected void notifyDataAvailability(String reason) { // note that we either just turned all off because we lost availability // or all were off and could now go on, so only have off apns to worry about - notifyOffApnsOfAvailability(reason, isDataPossible()); + notifyOffApnsOfAvailability(reason, isDataPossible(Phone.APN_TYPE_DEFAULT)); } public boolean isApnTypeEnabled(String apnType) { @@ -968,11 +967,7 @@ public abstract class DataConnectionTracker extends Handler { sendMessage(msg); } - public boolean isAnyActiveDataConnections() { - // TODO: Remember if there are any connected or - // loop asking each DC/APN? - return true; - } + public abstract boolean isAnyActiveDataConnections(); protected void onSetDataEnabled(boolean enable) { boolean prevEnabled = getAnyDataEnabled(); diff --git a/telephony/java/com/android/internal/telephony/PhoneBase.java b/telephony/java/com/android/internal/telephony/PhoneBase.java index b77e13426f6a2..f70d680fb6046 100644 --- a/telephony/java/com/android/internal/telephony/PhoneBase.java +++ b/telephony/java/com/android/internal/telephony/PhoneBase.java @@ -1023,7 +1023,7 @@ public abstract class PhoneBase extends Handler implements Phone { } public boolean isDataConnectivityPossible() { - return ((mDataConnectionTracker != null) && (mDataConnectionTracker.isDataPossible())); + return isDataConnectivityPossible(Phone.APN_TYPE_DEFAULT); } public boolean isDataConnectivityPossible(String apnType) { diff --git a/telephony/java/com/android/internal/telephony/PhoneProxy.java b/telephony/java/com/android/internal/telephony/PhoneProxy.java index 68f1c5fe24228..c2212dbae1af3 100644 --- a/telephony/java/com/android/internal/telephony/PhoneProxy.java +++ b/telephony/java/com/android/internal/telephony/PhoneProxy.java @@ -654,7 +654,7 @@ public class PhoneProxy extends Handler implements Phone { } public boolean isDataConnectivityPossible() { - return mActivePhone.isDataConnectivityPossible(); + return mActivePhone.isDataConnectivityPossible(Phone.APN_TYPE_DEFAULT); } public boolean isDataConnectivityPossible(String apnType) { diff --git a/telephony/java/com/android/internal/telephony/cdma/CdmaDataConnectionTracker.java b/telephony/java/com/android/internal/telephony/cdma/CdmaDataConnectionTracker.java index 8ce221a413def..800615cd0af71 100644 --- a/telephony/java/com/android/internal/telephony/cdma/CdmaDataConnectionTracker.java +++ b/telephony/java/com/android/internal/telephony/cdma/CdmaDataConnectionTracker.java @@ -214,35 +214,15 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker { return allowed; } - /** - * The only circumstances under which we report that data connectivity is not - * possible are - *