Merge "Revert "Skip handling validation status for non default data connections"" into qt-r1-dev (c2255338) · Commits · e / os / android_frameworks_opt_telephony

src/java/com/android/internal/telephony/dataconnection/DataConnection.java

+0 −10

Original line number	Diff line number	Diff line
		@@ -1297,16 +1297,6 @@ public class DataConnection extends StateMachine {
		return result;
		}

		/** @return {@code true} if validation is required, {@code false} otherwise. */
		public boolean isValidationRequired() {
		final NetworkCapabilities nc = getNetworkCapabilities();
		return nc != null
		&& nc.hasCapability(NetworkCapabilities.NET_CAPABILITY_INTERNET)
		&& nc.hasCapability(NetworkCapabilities.NET_CAPABILITY_NOT_RESTRICTED)
		&& nc.hasCapability(NetworkCapabilities.NET_CAPABILITY_TRUSTED)
		&& nc.hasCapability(NetworkCapabilities.NET_CAPABILITY_NOT_VPN);
		}

		/**
		* @return {@code True} if 464xlat should be skipped.
		*/

src/java/com/android/internal/telephony/dataconnection/DcNetworkAgent.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -170,7 +170,7 @@ public class DcNetworkAgent extends NetworkAgent {
		DcTracker dct = mPhone.getDcTracker(mTransportType);
		if (dct != null) {
		Message msg = dct.obtainMessage(DctConstants.EVENT_NETWORK_STATUS_CHANGED,
		status, mDataConnection.getCid(), redirectUrl);
		status, 0, redirectUrl);
		msg.sendToTarget();
		}
		}

src/java/com/android/internal/telephony/dataconnection/DcTracker.java

+3 −8

Original line number	Diff line number	Diff line
		@@ -2931,11 +2931,10 @@ public class DcTracker extends Handler {
		*
		* @param status One of {@code NetworkAgent.VALID_NETWORK} or
		* {@code NetworkAgent.INVALID_NETWORK}.
		* @param cid context id {@code cid}
		* @param redirectUrl If the Internet probe was redirected, this
		* is the destination it was redirected to, otherwise {@code null}
		*/
		private void onNetworkStatusChanged(int status, int cid, String redirectUrl) {
		private void onNetworkStatusChanged(int status, String redirectUrl) {
		if (!TextUtils.isEmpty(redirectUrl)) {
		Intent intent = new Intent(TelephonyIntents.ACTION_CARRIER_SIGNAL_REDIRECTED);
		intent.putExtra(TelephonyIntents.EXTRA_REDIRECTION_URL_KEY, redirectUrl);
		@@ -2943,7 +2942,6 @@ public class DcTracker extends Handler {
		log("Notify carrier signal receivers with redirectUrl: " + redirectUrl);
		} else {
		final boolean isValid = status == NetworkAgent.VALID_NETWORK;
		final DataConnection dc = getDataConnectionByContextId(cid);
		if (!mDsRecoveryHandler.isRecoveryOnBadNetworkEnabled()) {
		if (DBG) log("Skip data stall recovery on network status change with in threshold");
		return;
		@@ -2952,11 +2950,9 @@ public class DcTracker extends Handler {
		if (DBG) log("Skip data stall recovery on non WWAN");
		return;
		}
		if (dc != null && dc.isValidationRequired()) {
		mDsRecoveryHandler.processNetworkStatusChanged(isValid);
		}
		}
		}

		/**
		* Called when EVENT_DISCONNECT_DONE is received.
		@@ -3603,9 +3599,8 @@ public class DcTracker extends Handler {

		case DctConstants.EVENT_NETWORK_STATUS_CHANGED:
		int status = msg.arg1;
		int cid = msg.arg2;
		String url = (String) msg.obj;
		onNetworkStatusChanged(status, cid, url);
		onNetworkStatusChanged(status, url);
		break;

		case DctConstants.EVENT_RADIO_AVAILABLE:

src/java/com/android/internal/telephony/test/SimulatedCommands.java

+0 −9

Original line number	Diff line number	Diff line
		@@ -46,7 +46,6 @@ import android.telephony.Rlog;
		import android.telephony.ServiceState;
		import android.telephony.SignalStrength;
		import android.telephony.TelephonyManager;
		import android.telephony.data.ApnSetting;
		import android.telephony.data.DataCallResponse;
		import android.telephony.data.DataProfile;
		import android.telephony.emergency.EmergencyNumber;
		@@ -1175,14 +1174,6 @@ public class SimulatedCommands extends BaseCommands
		}
		}

		// Store different cids to simulate concurrent IMS and default data calls
		if ((dataProfile.getSupportedApnTypesBitmask() & ApnSetting.TYPE_IMS)
		== ApnSetting.TYPE_IMS) {
		mSetupDataCallResult.cid = 0;
		} else {
		mSetupDataCallResult.cid = 1;
		}

		DataCallResponse response = RIL.convertDataCallResult(mSetupDataCallResult);
		if (mDcSuccess) {
		resultSuccess(result, response);

tests/telephonytests/src/com/android/internal/telephony/dataconnection/DcTrackerTest.java

+7 −10

Original line number	Diff line number	Diff line
		@@ -1571,17 +1571,16 @@ public class DcTrackerTest extends TelephonyTest {
		eq(AccessNetworkType.EUTRAN), dpCaptor.capture(),
		eq(false), eq(false), eq(DataService.REQUEST_REASON_NORMAL), any(),
		any(Message.class));
		waitForMs(200);
		verifyDataProfile(dpCaptor.getValue(), FAKE_APN1, 0, 21, 1, NETWORK_TYPE_LTE_BITMASK);

		logd("Sending EVENT_NETWORK_STATUS_CHANGED");
		mDct.sendMessage(mDct.obtainMessage(DctConstants.EVENT_NETWORK_STATUS_CHANGED,
		NetworkAgent.VALID_NETWORK, 1, null));
		NetworkAgent.VALID_NETWORK, 0, null));
		waitForMs(200);

		logd("Sending EVENT_NETWORK_STATUS_CHANGED");
		mDct.sendMessage(mDct.obtainMessage(DctConstants.EVENT_NETWORK_STATUS_CHANGED,
		NetworkAgent.INVALID_NETWORK, 1, null));
		NetworkAgent.INVALID_NETWORK, 0, null));
		waitForMs(200);

		// Verify that its no-op when the new data stall detection feature is disabled
		@@ -1613,17 +1612,16 @@ public class DcTrackerTest extends TelephonyTest {
		eq(AccessNetworkType.EUTRAN), dpCaptor.capture(),
		eq(false), eq(false), eq(DataService.REQUEST_REASON_NORMAL), any(),
		any(Message.class));
		waitForMs(200);
		verifyDataProfile(dpCaptor.getValue(), FAKE_APN1, 0, 21, 1, NETWORK_TYPE_LTE_BITMASK);

		logd("Sending EVENT_NETWORK_STATUS_CHANGED");
		mDct.sendMessage(mDct.obtainMessage(DctConstants.EVENT_NETWORK_STATUS_CHANGED,
		NetworkAgent.VALID_NETWORK, 1, null));
		NetworkAgent.VALID_NETWORK, 0, null));
		waitForMs(200);

		logd("Sending EVENT_NETWORK_STATUS_CHANGED");
		mDct.sendMessage(mDct.obtainMessage(DctConstants.EVENT_NETWORK_STATUS_CHANGED,
		NetworkAgent.INVALID_NETWORK, 1, null));
		NetworkAgent.INVALID_NETWORK, 0, null));
		waitForMs(200);

		verify(mSimulatedCommandsVerifier, times(1)).getDataCallList(any(Message.class));
		@@ -1656,12 +1654,11 @@ public class DcTrackerTest extends TelephonyTest {
		eq(AccessNetworkType.EUTRAN), dpCaptor.capture(),
		eq(false), eq(false), eq(DataService.REQUEST_REASON_NORMAL), any(),
		any(Message.class));
		waitForMs(200);
		verifyDataProfile(dpCaptor.getValue(), FAKE_APN1, 0, 21, 1, NETWORK_TYPE_LTE_BITMASK);

		logd("Sending EVENT_NETWORK_STATUS_CHANGED false");
		mDct.sendMessage(mDct.obtainMessage(DctConstants.EVENT_NETWORK_STATUS_CHANGED,
		NetworkAgent.INVALID_NETWORK, 1, null));
		NetworkAgent.INVALID_NETWORK, 0, null));
		waitForMs(200);

		// expected tear down all DataConnections
		@@ -1701,7 +1698,7 @@ public class DcTrackerTest extends TelephonyTest {

		logd("Sending EVENT_NETWORK_STATUS_CHANGED false");
		mDct.sendMessage(mDct.obtainMessage(DctConstants.EVENT_NETWORK_STATUS_CHANGED,
		NetworkAgent.INVALID_NETWORK, 1, null));
		NetworkAgent.INVALID_NETWORK, 0, null));
		waitForMs(200);

		// expected to get preferred network type
		@@ -1738,7 +1735,7 @@ public class DcTrackerTest extends TelephonyTest {

		logd("Sending EVENT_NETWORK_STATUS_CHANGED false");
		mDct.sendMessage(mDct.obtainMessage(DctConstants.EVENT_NETWORK_STATUS_CHANGED,
		NetworkAgent.INVALID_NETWORK, 1, null));
		NetworkAgent.INVALID_NETWORK, 0, null));
		waitForMs(200);

		// expected to get preferred network type