Merge "Update state in NetworkInfo when network resumes" am: 9e3dd9a78f
Change-Id: I26e4b318f7acacde38ee5cf0977fd63fc5567d79
This commit is contained in:
@@ -6898,6 +6898,15 @@ public class ConnectivityService extends IConnectivityManager.Stub
|
||||
// worry about multiple different substates of CONNECTED.
|
||||
newInfo.setDetailedState(NetworkInfo.DetailedState.SUSPENDED, info.getReason(),
|
||||
info.getExtraInfo());
|
||||
} else if (!suspended && info.getDetailedState() == NetworkInfo.DetailedState.SUSPENDED) {
|
||||
// SUSPENDED state is currently only overridden from CONNECTED state. In the case the
|
||||
// network agent is created, then goes to suspended, then goes out of suspended without
|
||||
// ever setting connected. Check if network agent is ever connected to update the state.
|
||||
newInfo.setDetailedState(nai.everConnected
|
||||
? NetworkInfo.DetailedState.CONNECTED
|
||||
: NetworkInfo.DetailedState.CONNECTING,
|
||||
info.getReason(),
|
||||
info.getExtraInfo());
|
||||
}
|
||||
newInfo.setRoaming(!nai.networkCapabilities.hasCapability(NET_CAPABILITY_NOT_ROAMING));
|
||||
return newInfo;
|
||||
|
||||
@@ -3171,6 +3171,7 @@ public class ConnectivityServiceTest {
|
||||
mCellNetworkAgent);
|
||||
cellNetworkCallback.expectCallback(CallbackEntry.SUSPENDED, mCellNetworkAgent);
|
||||
cellNetworkCallback.assertNoCallback();
|
||||
assertEquals(NetworkInfo.State.SUSPENDED, mCm.getActiveNetworkInfo().getState());
|
||||
|
||||
// Register a garden variety default network request.
|
||||
TestNetworkCallback dfltNetworkCallback = new TestNetworkCallback();
|
||||
@@ -3186,6 +3187,7 @@ public class ConnectivityServiceTest {
|
||||
mCellNetworkAgent);
|
||||
cellNetworkCallback.expectCallback(CallbackEntry.RESUMED, mCellNetworkAgent);
|
||||
cellNetworkCallback.assertNoCallback();
|
||||
assertEquals(NetworkInfo.State.CONNECTED, mCm.getActiveNetworkInfo().getState());
|
||||
|
||||
dfltNetworkCallback = new TestNetworkCallback();
|
||||
mCm.registerDefaultNetworkCallback(dfltNetworkCallback);
|
||||
|
||||
Reference in New Issue
Block a user