am c693969: Merge "Remove isAnyActiveDataConnection and promote isDisconnected." into ics-factoryrom

* commit 'c69396973b8a496e298a7076842b0e34d193651e':
  Remove isAnyActiveDataConnection and promote isDisconnected.

Author: Wink Saville

telephony/java/com/android/internal/telephony/DataConnectionTracker.java

@@ -1019,7 +1019,7 @@ public void cleanUpAllConnections(String cause) {
         sendMessage(msg);
     }
 
-    public abstract boolean isAnyActiveDataConnections();
+    public abstract boolean isDisconnected();
 
     protected void onSetUserDataEnabled(boolean enabled) {
         synchronized (mDataEnabledLock) {

telephony/java/com/android/internal/telephony/ServiceStateTracker.java

@@ -403,7 +403,14 @@ public void unregisterForPsRestrictedDisabled(Handler h) {
     public void powerOffRadioSafely(DataConnectionTracker dcTracker) {
         synchronized (this) {
             if (!mPendingRadioPowerOffAfterDataOff) {
-                if (dcTracker.isAnyActiveDataConnections()) {
+                // To minimize race conditions we call cleanUpAllConnections on
+                // both if else paths instead of before this isDisconnected test.
+                if (dcTracker.isDisconnected()) {
+                    // To minimize race conditions we do this after isDisconnected
+                    dcTracker.cleanUpAllConnections(Phone.REASON_RADIO_TURNED_OFF);
+                    if (DBG) log("Data disconnected, turn off radio right away.");
+                    hangupAndPowerOff();
+                } else {
                     dcTracker.cleanUpAllConnections(Phone.REASON_RADIO_TURNED_OFF);
                     Message msg = Message.obtain(this);
                     msg.what = EVENT_SET_RADIO_POWER_OFF;
@@ -415,10 +422,6 @@ public void powerOffRadioSafely(DataConnectionTracker dcTracker) {
                         log("Cannot send delayed Msg, turn off radio right away.");
                         hangupAndPowerOff();
                     }
-                } else {
-                    dcTracker.cleanUpAllConnections(Phone.REASON_RADIO_TURNED_OFF);
-                    if (DBG) log("Data disconnected, turn off radio right away.");
-                    hangupAndPowerOff();
                 }
             }
         }

telephony/java/com/android/internal/telephony/cdma/CdmaDataConnectionTracker.java

@@ -946,8 +946,8 @@ public void handleMessage (Message msg) {
     }
 
     @Override
-    public boolean isAnyActiveDataConnections() {
-        return (mState != State.IDLE);
+    public boolean isDisconnected() {
+        return ((mState == State.IDLE) || (mState == State.FAILED));
     }
 
     @Override

telephony/java/com/android/internal/telephony/gsm/GsmDataConnectionTracker.java

@@ -1961,7 +1961,8 @@ protected boolean isConnected() {
         return false;
     }
 
-    protected boolean isDisconnected() {
+    @Override
+    public boolean isDisconnected() {
         for (ApnContext apnContext : mApnContexts.values()) {
             if (!apnContext.isDisconnected()) {
                 // At least one context was not disconnected return false
@@ -2347,11 +2348,6 @@ private int getCellLocationId() {
         return cid;
     }
 
-    @Override
-    public boolean isAnyActiveDataConnections() {
-        return isConnected();
-    }
-
     @Override
     protected void log(String s) {
         Log.d(LOG_TAG, "[GsmDCT] "+ s);