LineageOS/android_packages_modules_Connectivity
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge "Revert isDefaultNetworkActive behavior when there is no default network" into main

Browse Source
This commit is contained in:
Motomu Utsumi
2023-08-01 05:49:47 +00:00
committed by Gerrit Code Review
parent 13a35ea213 51e7a60359
commit dc9cb1da23
2 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
service/src/com/android/server/ConnectivityService.java
View File

@@ -11138,7 +11138,9 @@ public class ConnectivityService extends IConnectivityManager.Stub
new RemoteCallbackList<>(); new RemoteCallbackList<>();
// Indicate the current system default network activity is active or not. // Indicate the current system default network activity is active or not.
// This needs to be volatile to allow non handler threads to read this value without lock. // This needs to be volatile to allow non handler threads to read this value without lock.
private volatile boolean mIsDefaultNetworkActive; // If there is no default network, default network is considered active to keep the existing
// behavior. Initial value is used until first connect to the default network.
private volatile boolean mIsDefaultNetworkActive = true;
private final ArrayMap<String, IdleTimerParams> mActiveIdleTimers = new ArrayMap<>(); private final ArrayMap<String, IdleTimerParams> mActiveIdleTimers = new ArrayMap<>();
private static class IdleTimerParams { private static class IdleTimerParams {
@@ -11321,8 +11323,9 @@ public class ConnectivityService extends IConnectivityManager.Stub
reportNetworkActive(); reportNetworkActive();
} }
} else { } else {
// If there is no default network, default network is considered inactive. // If there is no default network, default network is considered active to keep the
mIsDefaultNetworkActive = false; // existing behavior.
mIsDefaultNetworkActive = true;
} }
} }

5
tests/unit/java/com/android/server/ConnectivityServiceTest.java
View File

@@ -11330,7 +11330,8 @@ public class ConnectivityServiceTest {
@Test @Test
public void testIsDefaultNetworkActiveNoDefaultNetwork() throws Exception { public void testIsDefaultNetworkActiveNoDefaultNetwork() throws Exception {
assertFalse(mCm.isDefaultNetworkActive()); // isDefaultNetworkActive returns true if there is no default network, which is known issue.
assertTrue(mCm.isDefaultNetworkActive());
final LinkProperties cellLp = new LinkProperties(); final LinkProperties cellLp = new LinkProperties();
cellLp.setInterfaceName(MOBILE_IFNAME); cellLp.setInterfaceName(MOBILE_IFNAME);
@@ -11342,7 +11343,7 @@ public class ConnectivityServiceTest {
mCellAgent.disconnect(); mCellAgent.disconnect();
waitForIdle(); waitForIdle();
assertFalse(mCm.isDefaultNetworkActive()); assertTrue(mCm.isDefaultNetworkActive());
} }
@Test @Test