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

Merge "Rename PREFERENCE_PRIORITY_* to PREFERENCE_ORDER_*"

Browse Source
This commit is contained in:
Paul Hu
2021-07-26 08:12:42 +00:00
committed by Gerrit Code Review
parent 02d7eb2abe 48291863d7
commit 0e3b1e0b57
2 changed files with 96 additions and 103 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -403,44 +403,44 @@ public class ConnectivityService extends IConnectivityManager.Stub
/** /**
* For per-app preferences, requests contain an int to signify which request * For per-app preferences, requests contain an int to signify which request
* should have priority. The priority is passed to netd which will use it * should have priority. The order is passed to netd which will use it together
* together with UID ranges to generate the corresponding IP rule. This serves * with UID ranges to generate the corresponding IP rule. This serves to
* to direct device-originated data traffic of the specific UIDs to the correct * direct device-originated data traffic of the specific UIDs to the correct
* default network for each app. * default network for each app.
* Priorities passed to netd must be in the 0~999 range. Larger values code for * Order ints passed to netd must be in the 0~999 range. Larger values code for
* a lower priority, {@see NativeUidRangeConfig} * a lower priority, {@see NativeUidRangeConfig}
* *
* Requests that don't code for a per-app preference use PREFERENCE_PRIORITY_INVALID. * Requests that don't code for a per-app preference use PREFERENCE_ORDER_INVALID.
* The default request uses PREFERENCE_PRIORITY_DEFAULT. * The default request uses PREFERENCE_ORDER_DEFAULT.
*/ */
// Bound for the lowest valid priority. // Bound for the lowest valid preference order.
static final int PREFERENCE_PRIORITY_LOWEST = 999; static final int PREFERENCE_ORDER_LOWEST = 999;
// Used when sending to netd to code for "no priority". // Used when sending to netd to code for "no order".
static final int PREFERENCE_PRIORITY_NONE = 0; static final int PREFERENCE_ORDER_NONE = 0;
// Priority for requests that don't code for a per-app preference. As it is // Order for requests that don't code for a per-app preference. As it is
// out of the valid range, the corresponding priority should be // out of the valid range, the corresponding order should be
// PREFERENCE_PRIORITY_NONE when sending to netd. // PREFERENCE_ORDER_NONE when sending to netd.
@VisibleForTesting @VisibleForTesting
static final int PREFERENCE_PRIORITY_INVALID = Integer.MAX_VALUE; static final int PREFERENCE_ORDER_INVALID = Integer.MAX_VALUE;
// Priority for the default internet request. Since this must always have the // Order for the default internet request. Since this must always have the
// lowest priority, its value is larger than the largest acceptable value. As // lowest priority, its value is larger than the largest acceptable value. As
// it is out of the valid range, the corresponding priority should be // it is out of the valid range, the corresponding order should be
// PREFERENCE_PRIORITY_NONE when sending to netd. // PREFERENCE_ORDER_NONE when sending to netd.
static final int PREFERENCE_PRIORITY_DEFAULT = 1000; static final int PREFERENCE_ORDER_DEFAULT = 1000;
// As a security feature, VPNs have the top priority. // As a security feature, VPNs have the top priority.
static final int PREFERENCE_PRIORITY_VPN = 0; // Netd supports only 0 for VPN. static final int PREFERENCE_ORDER_VPN = 0; // Netd supports only 0 for VPN.
// Priority of per-app OEM preference. See {@link #setOemNetworkPreference}. // Order of per-app OEM preference. See {@link #setOemNetworkPreference}.
@VisibleForTesting @VisibleForTesting
static final int PREFERENCE_PRIORITY_OEM = 10; static final int PREFERENCE_ORDER_OEM = 10;
// Priority of per-profile preference, such as used by enterprise networks. // Order of per-profile preference, such as used by enterprise networks.
// See {@link #setProfileNetworkPreference}. // See {@link #setProfileNetworkPreference}.
@VisibleForTesting @VisibleForTesting
static final int PREFERENCE_PRIORITY_PROFILE = 20; static final int PREFERENCE_ORDER_PROFILE = 20;
// Priority of user setting to prefer mobile data even when networks with // Order of user setting to prefer mobile data even when networks with
// better scores are connected. // better scores are connected.
// See {@link ConnectivitySettingsManager#setMobileDataPreferredUids} // See {@link ConnectivitySettingsManager#setMobileDataPreferredUids}
@VisibleForTesting @VisibleForTesting
static final int PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED = 30; static final int PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED = 30;
/** /**
* used internally to clear a wakelock when transitioning * used internally to clear a wakelock when transitioning
@@ -4245,7 +4245,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
mNetd.networkRemoveUidRangesParcel(new NativeUidRangeConfig( mNetd.networkRemoveUidRangesParcel(new NativeUidRangeConfig(
satisfier.network.getNetId(), satisfier.network.getNetId(),
toUidRangeStableParcels(nri.getUids()), toUidRangeStableParcels(nri.getUids()),
nri.getPriorityForNetd())); nri.getPreferenceOrderForNetd()));
} catch (RemoteException e) { } catch (RemoteException e) {
loge("Exception setting network preference default network", e); loge("Exception setting network preference default network", e);
} }
@@ -5714,8 +5714,8 @@ public class ConnectivityService extends IConnectivityManager.Stub
// maximum limit of registered callbacks per UID. // maximum limit of registered callbacks per UID.
final int mAsUid; final int mAsUid;
// Default network priority of this request. // Preference order of this request.
final int mPreferencePriority; final int mPreferenceOrder;
// In order to preserve the mapping of NetworkRequest-to-callback when apps register // In order to preserve the mapping of NetworkRequest-to-callback when apps register
// callbacks using a returned NetworkRequest, the original NetworkRequest needs to be // callbacks using a returned NetworkRequest, the original NetworkRequest needs to be
@@ -5747,12 +5747,12 @@ public class ConnectivityService extends IConnectivityManager.Stub
NetworkRequestInfo(int asUid, @NonNull final NetworkRequest r, NetworkRequestInfo(int asUid, @NonNull final NetworkRequest r,
@Nullable final PendingIntent pi, @Nullable String callingAttributionTag) { @Nullable final PendingIntent pi, @Nullable String callingAttributionTag) {
this(asUid, Collections.singletonList(r), r, pi, callingAttributionTag, this(asUid, Collections.singletonList(r), r, pi, callingAttributionTag,
PREFERENCE_PRIORITY_INVALID); PREFERENCE_ORDER_INVALID);
} }
NetworkRequestInfo(int asUid, @NonNull final List<NetworkRequest> r, NetworkRequestInfo(int asUid, @NonNull final List<NetworkRequest> r,
@NonNull final NetworkRequest requestForCallback, @Nullable final PendingIntent pi, @NonNull final NetworkRequest requestForCallback, @Nullable final PendingIntent pi,
@Nullable String callingAttributionTag, final int preferencePriority) { @Nullable String callingAttributionTag, final int preferenceOrder) {
ensureAllNetworkRequestsHaveType(r); ensureAllNetworkRequestsHaveType(r);
mRequests = initializeRequests(r); mRequests = initializeRequests(r);
mNetworkRequestForCallback = requestForCallback; mNetworkRequestForCallback = requestForCallback;
@@ -5770,7 +5770,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
*/ */
mCallbackFlags = NetworkCallback.FLAG_NONE; mCallbackFlags = NetworkCallback.FLAG_NONE;
mCallingAttributionTag = callingAttributionTag; mCallingAttributionTag = callingAttributionTag;
mPreferencePriority = preferencePriority; mPreferenceOrder = preferenceOrder;
} }
NetworkRequestInfo(int asUid, @NonNull final NetworkRequest r, @Nullable final Messenger m, NetworkRequestInfo(int asUid, @NonNull final NetworkRequest r, @Nullable final Messenger m,
@@ -5800,7 +5800,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
mPerUidCounter.incrementCountOrThrow(mUid); mPerUidCounter.incrementCountOrThrow(mUid);
mCallbackFlags = callbackFlags; mCallbackFlags = callbackFlags;
mCallingAttributionTag = callingAttributionTag; mCallingAttributionTag = callingAttributionTag;
mPreferencePriority = PREFERENCE_PRIORITY_INVALID; mPreferenceOrder = PREFERENCE_ORDER_INVALID;
linkDeathRecipient(); linkDeathRecipient();
} }
@@ -5840,18 +5840,18 @@ public class ConnectivityService extends IConnectivityManager.Stub
mPerUidCounter.incrementCountOrThrow(mUid); mPerUidCounter.incrementCountOrThrow(mUid);
mCallbackFlags = nri.mCallbackFlags; mCallbackFlags = nri.mCallbackFlags;
mCallingAttributionTag = nri.mCallingAttributionTag; mCallingAttributionTag = nri.mCallingAttributionTag;
mPreferencePriority = PREFERENCE_PRIORITY_INVALID; mPreferenceOrder = PREFERENCE_ORDER_INVALID;
linkDeathRecipient(); linkDeathRecipient();
} }
NetworkRequestInfo(int asUid, @NonNull final NetworkRequest r) { NetworkRequestInfo(int asUid, @NonNull final NetworkRequest r) {
this(asUid, Collections.singletonList(r), PREFERENCE_PRIORITY_INVALID); this(asUid, Collections.singletonList(r), PREFERENCE_ORDER_INVALID);
} }
NetworkRequestInfo(int asUid, @NonNull final List<NetworkRequest> r, NetworkRequestInfo(int asUid, @NonNull final List<NetworkRequest> r,
final int preferencePriority) { final int preferenceOrder) {
this(asUid, r, r.get(0), null /* pi */, null /* callingAttributionTag */, this(asUid, r, r.get(0), null /* pi */, null /* callingAttributionTag */,
preferencePriority); preferenceOrder);
} }
// True if this NRI is being satisfied. It also accounts for if the nri has its satisifer // True if this NRI is being satisfied. It also accounts for if the nri has its satisifer
@@ -5892,17 +5892,17 @@ public class ConnectivityService extends IConnectivityManager.Stub
} }
} }
boolean hasHigherPriorityThan(@NonNull final NetworkRequestInfo target) { boolean hasHigherOrderThan(@NonNull final NetworkRequestInfo target) {
// Compare two priorities, larger value means lower priority. // Compare two preference orders.
return mPreferencePriority < target.mPreferencePriority; return mPreferenceOrder < target.mPreferenceOrder;
} }
int getPriorityForNetd() { int getPreferenceOrderForNetd() {
if (mPreferencePriority >= PREFERENCE_PRIORITY_NONE if (mPreferenceOrder >= PREFERENCE_ORDER_NONE
&& mPreferencePriority <= PREFERENCE_PRIORITY_LOWEST) { && mPreferenceOrder <= PREFERENCE_ORDER_LOWEST) {
return mPreferencePriority; return mPreferenceOrder;
} }
return PREFERENCE_PRIORITY_NONE; return PREFERENCE_ORDER_NONE;
} }
@Override @Override
@@ -5922,7 +5922,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
+ " " + mRequests + " " + mRequests
+ (mPendingIntent == null ? "" : " to trigger " + mPendingIntent) + (mPendingIntent == null ? "" : " to trigger " + mPendingIntent)
+ " callback flags: " + mCallbackFlags + " callback flags: " + mCallbackFlags
+ " priority: " + mPreferencePriority; + " order: " + mPreferenceOrder;
} }
} }
@@ -6514,7 +6514,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
// than one request and for multilayer, all requests will track the same uids. // than one request and for multilayer, all requests will track the same uids.
if (nri.mRequests.get(0).networkCapabilities.appliesToUid(uid)) { if (nri.mRequests.get(0).networkCapabilities.appliesToUid(uid)) {
// Find out the highest priority request. // Find out the highest priority request.
if (nri.hasHigherPriorityThan(highestPriorityNri)) { if (nri.hasHigherOrderThan(highestPriorityNri)) {
highestPriorityNri = nri; highestPriorityNri = nri;
} }
} }
@@ -6659,7 +6659,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
} }
for (final UidRange range : uids) { for (final UidRange range : uids) {
if (range.contains(uid)) { if (range.contains(uid)) {
if (nri.hasHigherPriorityThan(highestPriorityNri)) { if (nri.hasHigherOrderThan(highestPriorityNri)) {
highestPriorityNri = nri; highestPriorityNri = nri;
} }
} }
@@ -7524,10 +7524,10 @@ public class ConnectivityService extends IConnectivityManager.Stub
try { try {
if (add) { if (add) {
mNetd.networkAddUidRangesParcel(new NativeUidRangeConfig( mNetd.networkAddUidRangesParcel(new NativeUidRangeConfig(
nai.network.netId, ranges, PREFERENCE_PRIORITY_VPN)); nai.network.netId, ranges, PREFERENCE_ORDER_VPN));
} else { } else {
mNetd.networkRemoveUidRangesParcel(new NativeUidRangeConfig( mNetd.networkRemoveUidRangesParcel(new NativeUidRangeConfig(
nai.network.netId, ranges, PREFERENCE_PRIORITY_VPN)); nai.network.netId, ranges, PREFERENCE_ORDER_VPN));
} }
} catch (Exception e) { } catch (Exception e) {
loge("Exception while " + (add ? "adding" : "removing") + " uid ranges " + uidRanges + loge("Exception while " + (add ? "adding" : "removing") + " uid ranges " + uidRanges +
@@ -7872,13 +7872,13 @@ public class ConnectivityService extends IConnectivityManager.Stub
mNetd.networkAddUidRangesParcel(new NativeUidRangeConfig( mNetd.networkAddUidRangesParcel(new NativeUidRangeConfig(
newDefaultNetwork.network.getNetId(), newDefaultNetwork.network.getNetId(),
toUidRangeStableParcels(nri.getUids()), toUidRangeStableParcels(nri.getUids()),
nri.getPriorityForNetd())); nri.getPreferenceOrderForNetd()));
} }
if (null != oldDefaultNetwork) { if (null != oldDefaultNetwork) {
mNetd.networkRemoveUidRangesParcel(new NativeUidRangeConfig( mNetd.networkRemoveUidRangesParcel(new NativeUidRangeConfig(
oldDefaultNetwork.network.getNetId(), oldDefaultNetwork.network.getNetId(),
toUidRangeStableParcels(nri.getUids()), toUidRangeStableParcels(nri.getUids()),
nri.getPriorityForNetd())); nri.getPreferenceOrderForNetd()));
} }
} catch (RemoteException | ServiceSpecificException e) { } catch (RemoteException | ServiceSpecificException e) {
loge("Exception setting app default network", e); loge("Exception setting app default network", e);
@@ -9978,7 +9978,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
TYPE_NONE, NetworkRequest.Type.TRACK_DEFAULT)); TYPE_NONE, NetworkRequest.Type.TRACK_DEFAULT));
setNetworkRequestUids(nrs, UidRange.fromIntRanges(pref.capabilities.getUids())); setNetworkRequestUids(nrs, UidRange.fromIntRanges(pref.capabilities.getUids()));
final NetworkRequestInfo nri = new NetworkRequestInfo(Process.myUid(), nrs, final NetworkRequestInfo nri = new NetworkRequestInfo(Process.myUid(), nrs,
PREFERENCE_PRIORITY_PROFILE); PREFERENCE_ORDER_PROFILE);
result.add(nri); result.add(nri);
} }
return result; return result;
@@ -9995,7 +9995,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
() -> { () -> {
final ArraySet<NetworkRequestInfo> nris = final ArraySet<NetworkRequestInfo> nris =
createNrisFromProfileNetworkPreferences(mProfileNetworkPreferences); createNrisFromProfileNetworkPreferences(mProfileNetworkPreferences);
replaceDefaultNetworkRequestsForPreference(nris, PREFERENCE_PRIORITY_PROFILE); replaceDefaultNetworkRequestsForPreference(nris, PREFERENCE_ORDER_PROFILE);
}); });
// Finally, rematch. // Finally, rematch.
rematchAllNetworksAndRequests(); rematchAllNetworksAndRequests();
@@ -10035,7 +10035,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
} }
setNetworkRequestUids(requests, ranges); setNetworkRequestUids(requests, ranges);
nris.add(new NetworkRequestInfo(Process.myUid(), requests, nris.add(new NetworkRequestInfo(Process.myUid(), requests,
PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED)); PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED));
return nris; return nris;
} }
@@ -10047,7 +10047,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
final ArraySet<NetworkRequestInfo> nris = final ArraySet<NetworkRequestInfo> nris =
createNrisFromMobileDataPreferredUids(mMobileDataPreferredUids); createNrisFromMobileDataPreferredUids(mMobileDataPreferredUids);
replaceDefaultNetworkRequestsForPreference(nris, replaceDefaultNetworkRequestsForPreference(nris,
PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED); PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED);
}); });
// Finally, rematch. // Finally, rematch.
rematchAllNetworksAndRequests(); rematchAllNetworksAndRequests();
@@ -10145,7 +10145,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
final ArraySet<NetworkRequestInfo> nris = final ArraySet<NetworkRequestInfo> nris =
new OemNetworkRequestFactory() new OemNetworkRequestFactory()
.createNrisFromOemNetworkPreferences(preference); .createNrisFromOemNetworkPreferences(preference);
replaceDefaultNetworkRequestsForPreference(nris, PREFERENCE_PRIORITY_OEM); replaceDefaultNetworkRequestsForPreference(nris, PREFERENCE_ORDER_OEM);
}); });
mOemNetworkPreferences = preference; mOemNetworkPreferences = preference;
@@ -10159,11 +10159,11 @@ public class ConnectivityService extends IConnectivityManager.Stub
} }
private void replaceDefaultNetworkRequestsForPreference( private void replaceDefaultNetworkRequestsForPreference(
@NonNull final Set<NetworkRequestInfo> nris, final int preferencePriority) { @NonNull final Set<NetworkRequestInfo> nris, final int preferenceOrder) {
// Skip the requests which are set by other network preference. Because the uid range rules // Skip the requests which are set by other network preference. Because the uid range rules
// should stay in netd. // should stay in netd.
final Set<NetworkRequestInfo> requests = new ArraySet<>(mDefaultNetworkRequests); final Set<NetworkRequestInfo> requests = new ArraySet<>(mDefaultNetworkRequests);
requests.removeIf(request -> request.mPreferencePriority != preferencePriority); requests.removeIf(request -> request.mPreferenceOrder != preferenceOrder);
handleRemoveNetworkRequests(requests); handleRemoveNetworkRequests(requests);
addPerAppDefaultNetworkRequests(nris); addPerAppDefaultNetworkRequests(nris);
} }
@@ -10358,7 +10358,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
ranges.add(new UidRange(uid, uid)); ranges.add(new UidRange(uid, uid));
} }
setNetworkRequestUids(requests, ranges); setNetworkRequestUids(requests, ranges);
return new NetworkRequestInfo(Process.myUid(), requests, PREFERENCE_PRIORITY_OEM); return new NetworkRequestInfo(Process.myUid(), requests, PREFERENCE_ORDER_OEM);
} }
private NetworkRequest createUnmeteredNetworkRequest() { private NetworkRequest createUnmeteredNetworkRequest() {

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

@@ -125,10 +125,10 @@ import static android.net.resolv.aidl.IDnsResolverUnsolicitedEventListener.VALID
import static android.os.Process.INVALID_UID; import static android.os.Process.INVALID_UID;
import static android.system.OsConstants.IPPROTO_TCP; import static android.system.OsConstants.IPPROTO_TCP;
import static com.android.server.ConnectivityService.PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED; import static com.android.server.ConnectivityService.PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED;
import static com.android.server.ConnectivityService.PREFERENCE_PRIORITY_OEM; import static com.android.server.ConnectivityService.PREFERENCE_ORDER_OEM;
import static com.android.server.ConnectivityService.PREFERENCE_PRIORITY_PROFILE; import static com.android.server.ConnectivityService.PREFERENCE_ORDER_PROFILE;
import static com.android.server.ConnectivityService.PREFERENCE_PRIORITY_VPN; import static com.android.server.ConnectivityService.PREFERENCE_ORDER_VPN;
import static com.android.server.ConnectivityServiceTestUtils.transportToLegacyType; import static com.android.server.ConnectivityServiceTestUtils.transportToLegacyType;
import static com.android.testutils.ConcurrentUtils.await; import static com.android.testutils.ConcurrentUtils.await;
import static com.android.testutils.ConcurrentUtils.durationOf; import static com.android.testutils.ConcurrentUtils.durationOf;
@@ -1297,10 +1297,10 @@ public class ConnectivityServiceTest {
verify(mMockNetd, times(1)).networkAddUidRangesParcel( verify(mMockNetd, times(1)).networkAddUidRangesParcel(
new NativeUidRangeConfig(mMockVpn.getNetwork().getNetId(), new NativeUidRangeConfig(mMockVpn.getNetwork().getNetId(),
toUidRangeStableParcels(uids), PREFERENCE_PRIORITY_VPN)); toUidRangeStableParcels(uids), PREFERENCE_ORDER_VPN));
verify(mMockNetd, never()).networkRemoveUidRangesParcel(argThat(config -> verify(mMockNetd, never()).networkRemoveUidRangesParcel(argThat(config ->
mMockVpn.getNetwork().getNetId() == config.netId mMockVpn.getNetwork().getNetId() == config.netId
&& PREFERENCE_PRIORITY_VPN == config.subPriority)); && PREFERENCE_ORDER_VPN == config.subPriority));
mAgentRegistered = true; mAgentRegistered = true;
verify(mMockNetd).networkCreate(nativeNetworkConfigVpn(getNetwork().netId, verify(mMockNetd).networkCreate(nativeNetworkConfigVpn(getNetwork().netId,
!mMockNetworkAgent.isBypassableVpn(), mVpnType)); !mMockNetworkAgent.isBypassableVpn(), mVpnType));
@@ -10478,11 +10478,11 @@ public class ConnectivityServiceTest {
if (add) { if (add) {
inOrder.verify(mMockNetd, times(1)).networkAddUidRangesParcel( inOrder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(
new NativeUidRangeConfig(mMockVpn.getNetwork().getNetId(), new NativeUidRangeConfig(mMockVpn.getNetwork().getNetId(),
toUidRangeStableParcels(vpnRanges), PREFERENCE_PRIORITY_VPN)); toUidRangeStableParcels(vpnRanges), PREFERENCE_ORDER_VPN));
} else { } else {
inOrder.verify(mMockNetd, times(1)).networkRemoveUidRangesParcel( inOrder.verify(mMockNetd, times(1)).networkRemoveUidRangesParcel(
new NativeUidRangeConfig(mMockVpn.getNetwork().getNetId(), new NativeUidRangeConfig(mMockVpn.getNetwork().getNetId(),
toUidRangeStableParcels(vpnRanges), PREFERENCE_PRIORITY_VPN)); toUidRangeStableParcels(vpnRanges), PREFERENCE_ORDER_VPN));
} }
inOrder.verify(mMockNetd, times(1)).socketDestroy(eq(toUidRangeStableParcels(vpnRanges)), inOrder.verify(mMockNetd, times(1)).socketDestroy(eq(toUidRangeStableParcels(vpnRanges)),
@@ -10956,7 +10956,7 @@ public class ConnectivityServiceTest {
.createNrisFromOemNetworkPreferences( .createNrisFromOemNetworkPreferences(
createDefaultOemNetworkPreferences(prefToTest)); createDefaultOemNetworkPreferences(prefToTest));
final NetworkRequestInfo nri = nris.iterator().next(); final NetworkRequestInfo nri = nris.iterator().next();
assertEquals(PREFERENCE_PRIORITY_OEM, nri.mPreferencePriority); assertEquals(PREFERENCE_ORDER_OEM, nri.mPreferenceOrder);
final List<NetworkRequest> mRequests = nri.mRequests; final List<NetworkRequest> mRequests = nri.mRequests;
assertEquals(expectedNumOfNris, nris.size()); assertEquals(expectedNumOfNris, nris.size());
assertEquals(expectedNumOfRequests, mRequests.size()); assertEquals(expectedNumOfRequests, mRequests.size());
@@ -10986,7 +10986,7 @@ public class ConnectivityServiceTest {
.createNrisFromOemNetworkPreferences( .createNrisFromOemNetworkPreferences(
createDefaultOemNetworkPreferences(prefToTest)); createDefaultOemNetworkPreferences(prefToTest));
final NetworkRequestInfo nri = nris.iterator().next(); final NetworkRequestInfo nri = nris.iterator().next();
assertEquals(PREFERENCE_PRIORITY_OEM, nri.mPreferencePriority); assertEquals(PREFERENCE_ORDER_OEM, nri.mPreferenceOrder);
final List<NetworkRequest> mRequests = nri.mRequests; final List<NetworkRequest> mRequests = nri.mRequests;
assertEquals(expectedNumOfNris, nris.size()); assertEquals(expectedNumOfNris, nris.size());
assertEquals(expectedNumOfRequests, mRequests.size()); assertEquals(expectedNumOfRequests, mRequests.size());
@@ -11013,7 +11013,7 @@ public class ConnectivityServiceTest {
.createNrisFromOemNetworkPreferences( .createNrisFromOemNetworkPreferences(
createDefaultOemNetworkPreferences(prefToTest)); createDefaultOemNetworkPreferences(prefToTest));
final NetworkRequestInfo nri = nris.iterator().next(); final NetworkRequestInfo nri = nris.iterator().next();
assertEquals(PREFERENCE_PRIORITY_OEM, nri.mPreferencePriority); assertEquals(PREFERENCE_ORDER_OEM, nri.mPreferenceOrder);
final List<NetworkRequest> mRequests = nri.mRequests; final List<NetworkRequest> mRequests = nri.mRequests;
assertEquals(expectedNumOfNris, nris.size()); assertEquals(expectedNumOfNris, nris.size());
assertEquals(expectedNumOfRequests, mRequests.size()); assertEquals(expectedNumOfRequests, mRequests.size());
@@ -11037,7 +11037,7 @@ public class ConnectivityServiceTest {
.createNrisFromOemNetworkPreferences( .createNrisFromOemNetworkPreferences(
createDefaultOemNetworkPreferences(prefToTest)); createDefaultOemNetworkPreferences(prefToTest));
final NetworkRequestInfo nri = nris.iterator().next(); final NetworkRequestInfo nri = nris.iterator().next();
assertEquals(PREFERENCE_PRIORITY_OEM, nri.mPreferencePriority); assertEquals(PREFERENCE_ORDER_OEM, nri.mPreferenceOrder);
final List<NetworkRequest> mRequests = nri.mRequests; final List<NetworkRequest> mRequests = nri.mRequests;
assertEquals(expectedNumOfNris, nris.size()); assertEquals(expectedNumOfNris, nris.size());
assertEquals(expectedNumOfRequests, mRequests.size()); assertEquals(expectedNumOfRequests, mRequests.size());
@@ -11728,11 +11728,11 @@ public class ConnectivityServiceTest {
verify(mMockNetd, times(addUidRangesTimes)).networkAddUidRangesParcel(argThat(config -> verify(mMockNetd, times(addUidRangesTimes)).networkAddUidRangesParcel(argThat(config ->
(useAnyIdForAdd ? true : addUidRangesNetId == config.netId) (useAnyIdForAdd ? true : addUidRangesNetId == config.netId)
&& Arrays.equals(addedUidRanges, config.uidRanges) && Arrays.equals(addedUidRanges, config.uidRanges)
&& PREFERENCE_PRIORITY_OEM == config.subPriority)); && PREFERENCE_ORDER_OEM == config.subPriority));
verify(mMockNetd, times(removeUidRangesTimes)).networkRemoveUidRangesParcel( verify(mMockNetd, times(removeUidRangesTimes)).networkRemoveUidRangesParcel(
argThat(config -> (useAnyIdForRemove ? true : removeUidRangesNetId == config.netId) argThat(config -> (useAnyIdForRemove ? true : removeUidRangesNetId == config.netId)
&& Arrays.equals(removedUidRanges, config.uidRanges) && Arrays.equals(removedUidRanges, config.uidRanges)
&& PREFERENCE_PRIORITY_OEM == config.subPriority)); && PREFERENCE_ORDER_OEM == config.subPriority));
if (shouldDestroyNetwork) { if (shouldDestroyNetwork) {
verify(mMockNetd, times(1)) verify(mMockNetd, times(1))
.networkDestroy((useAnyIdForRemove ? anyInt() : eq(removeUidRangesNetId))); .networkDestroy((useAnyIdForRemove ? anyInt() : eq(removeUidRangesNetId)));
@@ -12887,7 +12887,7 @@ public class ConnectivityServiceTest {
// is not handled specially, the rules are always active as long as a preference is set. // is not handled specially, the rules are always active as long as a preference is set.
inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig(
mCellNetworkAgent.getNetwork().netId, uidRangeFor(testHandle), mCellNetworkAgent.getNetwork().netId, uidRangeFor(testHandle),
PREFERENCE_PRIORITY_PROFILE)); PREFERENCE_ORDER_PROFILE));
// The enterprise network is not ready yet. // The enterprise network is not ready yet.
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback, assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback,
@@ -12902,11 +12902,10 @@ public class ConnectivityServiceTest {
inOrder.verify(mMockNetd).networkCreate( inOrder.verify(mMockNetd).networkCreate(
nativeNetworkConfigPhysical(workAgent.getNetwork().netId, INetd.PERMISSION_SYSTEM)); nativeNetworkConfigPhysical(workAgent.getNetwork().netId, INetd.PERMISSION_SYSTEM));
inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig(
workAgent.getNetwork().netId, uidRangeFor(testHandle), workAgent.getNetwork().netId, uidRangeFor(testHandle), PREFERENCE_ORDER_PROFILE));
PREFERENCE_PRIORITY_PROFILE));
inOrder.verify(mMockNetd).networkRemoveUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkRemoveUidRangesParcel(new NativeUidRangeConfig(
mCellNetworkAgent.getNetwork().netId, uidRangeFor(testHandle), mCellNetworkAgent.getNetwork().netId, uidRangeFor(testHandle),
PREFERENCE_PRIORITY_PROFILE)); PREFERENCE_ORDER_PROFILE));
// Make sure changes to the work agent send callbacks to the app in the work profile, but // Make sure changes to the work agent send callbacks to the app in the work profile, but
// not to the other apps. // not to the other apps.
@@ -12956,7 +12955,7 @@ public class ConnectivityServiceTest {
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback); assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback);
inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig(
mCellNetworkAgent.getNetwork().netId, uidRangeFor(testHandle), mCellNetworkAgent.getNetwork().netId, uidRangeFor(testHandle),
PREFERENCE_PRIORITY_PROFILE)); PREFERENCE_ORDER_PROFILE));
inOrder.verify(mMockNetd).networkDestroy(workAgent.getNetwork().netId); inOrder.verify(mMockNetd).networkDestroy(workAgent.getNetwork().netId);
mCellNetworkAgent.disconnect(); mCellNetworkAgent.disconnect();
@@ -12980,8 +12979,7 @@ public class ConnectivityServiceTest {
inOrder.verify(mMockNetd).networkCreate(nativeNetworkConfigPhysical( inOrder.verify(mMockNetd).networkCreate(nativeNetworkConfigPhysical(
workAgent2.getNetwork().netId, INetd.PERMISSION_SYSTEM)); workAgent2.getNetwork().netId, INetd.PERMISSION_SYSTEM));
inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig(
workAgent2.getNetwork().netId, uidRangeFor(testHandle), workAgent2.getNetwork().netId, uidRangeFor(testHandle), PREFERENCE_ORDER_PROFILE));
PREFERENCE_PRIORITY_PROFILE));
workAgent2.setNetworkValid(true /* isStrictMode */); workAgent2.setNetworkValid(true /* isStrictMode */);
workAgent2.mNetworkMonitor.forceReevaluation(Process.myUid()); workAgent2.mNetworkMonitor.forceReevaluation(Process.myUid());
@@ -13027,8 +13025,7 @@ public class ConnectivityServiceTest {
inOrder.verify(mMockNetd).networkCreate(nativeNetworkConfigPhysical( inOrder.verify(mMockNetd).networkCreate(nativeNetworkConfigPhysical(
mCellNetworkAgent.getNetwork().netId, INetd.PERMISSION_NONE)); mCellNetworkAgent.getNetwork().netId, INetd.PERMISSION_NONE));
inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig(
workAgent.getNetwork().netId, uidRangeFor(testHandle), workAgent.getNetwork().netId, uidRangeFor(testHandle), PREFERENCE_ORDER_PROFILE));
PREFERENCE_PRIORITY_PROFILE));
registerDefaultNetworkCallbacks(); registerDefaultNetworkCallbacks();
@@ -13043,8 +13040,7 @@ public class ConnectivityServiceTest {
mProfileDefaultNetworkCallback.expectAvailableCallbacksValidated(mCellNetworkAgent); mProfileDefaultNetworkCallback.expectAvailableCallbacksValidated(mCellNetworkAgent);
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback); assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback);
inOrder.verify(mMockNetd).networkRemoveUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkRemoveUidRangesParcel(new NativeUidRangeConfig(
workAgent.getNetwork().netId, uidRangeFor(testHandle), workAgent.getNetwork().netId, uidRangeFor(testHandle), PREFERENCE_ORDER_PROFILE));
PREFERENCE_PRIORITY_PROFILE));
workAgent.disconnect(); workAgent.disconnect();
mCellNetworkAgent.disconnect(); mCellNetworkAgent.disconnect();
@@ -13089,8 +13085,7 @@ public class ConnectivityServiceTest {
r -> r.run(), listener); r -> r.run(), listener);
listener.expectOnComplete(); listener.expectOnComplete();
inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig(
workAgent.getNetwork().netId, uidRangeFor(testHandle2), workAgent.getNetwork().netId, uidRangeFor(testHandle2), PREFERENCE_ORDER_PROFILE));
PREFERENCE_PRIORITY_PROFILE));
mProfileDefaultNetworkCallback.expectAvailableCallbacksValidated(workAgent); mProfileDefaultNetworkCallback.expectAvailableCallbacksValidated(workAgent);
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback, assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback,
@@ -13100,8 +13095,7 @@ public class ConnectivityServiceTest {
r -> r.run(), listener); r -> r.run(), listener);
listener.expectOnComplete(); listener.expectOnComplete();
inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig(
workAgent.getNetwork().netId, uidRangeFor(testHandle4), workAgent.getNetwork().netId, uidRangeFor(testHandle4), PREFERENCE_ORDER_PROFILE));
PREFERENCE_PRIORITY_PROFILE));
app4Cb.expectAvailableCallbacksValidated(workAgent); app4Cb.expectAvailableCallbacksValidated(workAgent);
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback, assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback,
@@ -13111,8 +13105,7 @@ public class ConnectivityServiceTest {
r -> r.run(), listener); r -> r.run(), listener);
listener.expectOnComplete(); listener.expectOnComplete();
inOrder.verify(mMockNetd).networkRemoveUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkRemoveUidRangesParcel(new NativeUidRangeConfig(
workAgent.getNetwork().netId, uidRangeFor(testHandle2), workAgent.getNetwork().netId, uidRangeFor(testHandle2), PREFERENCE_ORDER_PROFILE));
PREFERENCE_PRIORITY_PROFILE));
mProfileDefaultNetworkCallback.expectAvailableCallbacksValidated(mCellNetworkAgent); mProfileDefaultNetworkCallback.expectAvailableCallbacksValidated(mCellNetworkAgent);
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback, assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback,
@@ -13141,7 +13134,7 @@ public class ConnectivityServiceTest {
mCellNetworkAgent.getNetwork().netId, INetd.PERMISSION_NONE)); mCellNetworkAgent.getNetwork().netId, INetd.PERMISSION_NONE));
inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkAddUidRangesParcel(new NativeUidRangeConfig(
mCellNetworkAgent.getNetwork().netId, uidRangeFor(testHandle), mCellNetworkAgent.getNetwork().netId, uidRangeFor(testHandle),
PREFERENCE_PRIORITY_PROFILE)); PREFERENCE_ORDER_PROFILE));
final Intent removedIntent = new Intent(ACTION_USER_REMOVED); final Intent removedIntent = new Intent(ACTION_USER_REMOVED);
removedIntent.putExtra(Intent.EXTRA_USER, testHandle); removedIntent.putExtra(Intent.EXTRA_USER, testHandle);
@@ -13149,7 +13142,7 @@ public class ConnectivityServiceTest {
inOrder.verify(mMockNetd).networkRemoveUidRangesParcel(new NativeUidRangeConfig( inOrder.verify(mMockNetd).networkRemoveUidRangesParcel(new NativeUidRangeConfig(
mCellNetworkAgent.getNetwork().netId, uidRangeFor(testHandle), mCellNetworkAgent.getNetwork().netId, uidRangeFor(testHandle),
PREFERENCE_PRIORITY_PROFILE)); PREFERENCE_ORDER_PROFILE));
} }
/** /**
@@ -13320,7 +13313,7 @@ public class ConnectivityServiceTest {
assertEquals(1, nris.size()); assertEquals(1, nris.size());
assertTrue(nri.isMultilayerRequest()); assertTrue(nri.isMultilayerRequest());
assertEquals(nri.getUids(), uidRangesForUids(uids)); assertEquals(nri.getUids(), uidRangesForUids(uids));
assertEquals(PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED, nri.mPreferencePriority); assertEquals(PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED, nri.mPreferenceOrder);
} }
/** /**
@@ -13372,7 +13365,7 @@ public class ConnectivityServiceTest {
final Set<Integer> uids1 = Set.of(PRIMARY_USER_HANDLE.getUid(TEST_PACKAGE_UID)); final Set<Integer> uids1 = Set.of(PRIMARY_USER_HANDLE.getUid(TEST_PACKAGE_UID));
final UidRangeParcel[] uidRanges1 = toUidRangeStableParcels(uidRangesForUids(uids1)); final UidRangeParcel[] uidRanges1 = toUidRangeStableParcels(uidRangesForUids(uids1));
final NativeUidRangeConfig config1 = new NativeUidRangeConfig(cellNetId, uidRanges1, final NativeUidRangeConfig config1 = new NativeUidRangeConfig(cellNetId, uidRanges1,
PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED); PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED);
setAndUpdateMobileDataPreferredUids(uids1); setAndUpdateMobileDataPreferredUids(uids1);
inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config1); inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config1);
inorder.verify(mMockNetd, never()).networkRemoveUidRangesParcel(any()); inorder.verify(mMockNetd, never()).networkRemoveUidRangesParcel(any());
@@ -13384,7 +13377,7 @@ public class ConnectivityServiceTest {
SECONDARY_USER_HANDLE.getUid(TEST_PACKAGE_UID)); SECONDARY_USER_HANDLE.getUid(TEST_PACKAGE_UID));
final UidRangeParcel[] uidRanges2 = toUidRangeStableParcels(uidRangesForUids(uids2)); final UidRangeParcel[] uidRanges2 = toUidRangeStableParcels(uidRangesForUids(uids2));
final NativeUidRangeConfig config2 = new NativeUidRangeConfig(cellNetId, uidRanges2, final NativeUidRangeConfig config2 = new NativeUidRangeConfig(cellNetId, uidRanges2,
PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED); PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED);
setAndUpdateMobileDataPreferredUids(uids2); setAndUpdateMobileDataPreferredUids(uids2);
inorder.verify(mMockNetd, times(1)).networkRemoveUidRangesParcel(config1); inorder.verify(mMockNetd, times(1)).networkRemoveUidRangesParcel(config1);
inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config2); inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config2);
@@ -13432,7 +13425,7 @@ public class ConnectivityServiceTest {
final Set<Integer> uids = Set.of(PRIMARY_USER_HANDLE.getUid(TEST_PACKAGE_UID)); final Set<Integer> uids = Set.of(PRIMARY_USER_HANDLE.getUid(TEST_PACKAGE_UID));
final UidRangeParcel[] uidRanges = toUidRangeStableParcels(uidRangesForUids(uids)); final UidRangeParcel[] uidRanges = toUidRangeStableParcels(uidRangesForUids(uids));
final NativeUidRangeConfig wifiConfig = new NativeUidRangeConfig(wifiNetId, uidRanges, final NativeUidRangeConfig wifiConfig = new NativeUidRangeConfig(wifiNetId, uidRanges,
PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED); PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED);
setAndUpdateMobileDataPreferredUids(uids); setAndUpdateMobileDataPreferredUids(uids);
inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(wifiConfig); inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(wifiConfig);
inorder.verify(mMockNetd, never()).networkRemoveUidRangesParcel(any()); inorder.verify(mMockNetd, never()).networkRemoveUidRangesParcel(any());
@@ -13448,7 +13441,7 @@ public class ConnectivityServiceTest {
final int cellNetId = mCellNetworkAgent.getNetwork().netId; final int cellNetId = mCellNetworkAgent.getNetwork().netId;
final NativeUidRangeConfig cellConfig = new NativeUidRangeConfig(cellNetId, uidRanges, final NativeUidRangeConfig cellConfig = new NativeUidRangeConfig(cellNetId, uidRanges,
PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED); PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED);
inorder.verify(mMockNetd, times(1)).networkCreate(nativeNetworkConfigPhysical( inorder.verify(mMockNetd, times(1)).networkCreate(nativeNetworkConfigPhysical(
cellNetId, INetd.PERMISSION_NONE)); cellNetId, INetd.PERMISSION_NONE));
inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(cellConfig); inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(cellConfig);
@@ -13477,7 +13470,7 @@ public class ConnectivityServiceTest {
final int cellNetId2 = mCellNetworkAgent.getNetwork().netId; final int cellNetId2 = mCellNetworkAgent.getNetwork().netId;
final NativeUidRangeConfig cellConfig2 = new NativeUidRangeConfig(cellNetId2, uidRanges, final NativeUidRangeConfig cellConfig2 = new NativeUidRangeConfig(cellNetId2, uidRanges,
PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED); PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED);
inorder.verify(mMockNetd, times(1)).networkCreate(nativeNetworkConfigPhysical( inorder.verify(mMockNetd, times(1)).networkCreate(nativeNetworkConfigPhysical(
cellNetId2, INetd.PERMISSION_NONE)); cellNetId2, INetd.PERMISSION_NONE));
inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(cellConfig2); inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(cellConfig2);
@@ -13576,7 +13569,7 @@ public class ConnectivityServiceTest {
final int[] uids1 = new int[] { PRIMARY_USER_HANDLE.getUid(TEST_PACKAGE_UID) }; final int[] uids1 = new int[] { PRIMARY_USER_HANDLE.getUid(TEST_PACKAGE_UID) };
final UidRangeParcel[] uidRanges1 = toUidRangeStableParcels(uidRangesForUids(uids1)); final UidRangeParcel[] uidRanges1 = toUidRangeStableParcels(uidRangesForUids(uids1));
final NativeUidRangeConfig config1 = new NativeUidRangeConfig(cellNetId, uidRanges1, final NativeUidRangeConfig config1 = new NativeUidRangeConfig(cellNetId, uidRanges1,
PREFERENCE_PRIORITY_OEM); PREFERENCE_ORDER_OEM);
setupSetOemNetworkPreferenceForPreferenceTest(networkPref, uidRanges1, TEST_PACKAGE_NAME); setupSetOemNetworkPreferenceForPreferenceTest(networkPref, uidRanges1, TEST_PACKAGE_NAME);
inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config1); inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config1);
inorder.verify(mMockNetd, never()).networkRemoveUidRangesParcel(any()); inorder.verify(mMockNetd, never()).networkRemoveUidRangesParcel(any());
@@ -13590,7 +13583,7 @@ public class ConnectivityServiceTest {
r -> r.run(), listener); r -> r.run(), listener);
listener.expectOnComplete(); listener.expectOnComplete();
final NativeUidRangeConfig config2 = new NativeUidRangeConfig(workAgent.getNetwork().netId, final NativeUidRangeConfig config2 = new NativeUidRangeConfig(workAgent.getNetwork().netId,
uidRangeFor(testHandle), PREFERENCE_PRIORITY_PROFILE); uidRangeFor(testHandle), PREFERENCE_ORDER_PROFILE);
inorder.verify(mMockNetd).networkCreate(nativeNetworkConfigPhysical( inorder.verify(mMockNetd).networkCreate(nativeNetworkConfigPhysical(
workAgent.getNetwork().netId, INetd.PERMISSION_SYSTEM)); workAgent.getNetwork().netId, INetd.PERMISSION_SYSTEM));
inorder.verify(mMockNetd, never()).networkRemoveUidRangesParcel(any()); inorder.verify(mMockNetd, never()).networkRemoveUidRangesParcel(any());
@@ -13600,7 +13593,7 @@ public class ConnectivityServiceTest {
final Set<Integer> uids2 = Set.of(PRIMARY_USER_HANDLE.getUid(TEST_PACKAGE_UID2)); final Set<Integer> uids2 = Set.of(PRIMARY_USER_HANDLE.getUid(TEST_PACKAGE_UID2));
final UidRangeParcel[] uidRanges2 = toUidRangeStableParcels(uidRangesForUids(uids2)); final UidRangeParcel[] uidRanges2 = toUidRangeStableParcels(uidRangesForUids(uids2));
final NativeUidRangeConfig config3 = new NativeUidRangeConfig(cellNetId, uidRanges2, final NativeUidRangeConfig config3 = new NativeUidRangeConfig(cellNetId, uidRanges2,
PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED); PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED);
setAndUpdateMobileDataPreferredUids(uids2); setAndUpdateMobileDataPreferredUids(uids2);
inorder.verify(mMockNetd, never()).networkRemoveUidRangesParcel(any()); inorder.verify(mMockNetd, never()).networkRemoveUidRangesParcel(any());
inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config3); inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config3);
@@ -13609,7 +13602,7 @@ public class ConnectivityServiceTest {
final Set<Integer> uids3 = Set.of(PRIMARY_USER_HANDLE.getUid(TEST_PACKAGE_UID3)); final Set<Integer> uids3 = Set.of(PRIMARY_USER_HANDLE.getUid(TEST_PACKAGE_UID3));
final UidRangeParcel[] uidRanges3 = toUidRangeStableParcels(uidRangesForUids(uids3)); final UidRangeParcel[] uidRanges3 = toUidRangeStableParcels(uidRangesForUids(uids3));
final NativeUidRangeConfig config4 = new NativeUidRangeConfig(cellNetId, uidRanges3, final NativeUidRangeConfig config4 = new NativeUidRangeConfig(cellNetId, uidRanges3,
PREFERENCE_PRIORITY_OEM); PREFERENCE_ORDER_OEM);
setupSetOemNetworkPreferenceForPreferenceTest(networkPref, uidRanges3, "com.android.test"); setupSetOemNetworkPreferenceForPreferenceTest(networkPref, uidRanges3, "com.android.test");
inorder.verify(mMockNetd, times(1)).networkRemoveUidRangesParcel(config1); inorder.verify(mMockNetd, times(1)).networkRemoveUidRangesParcel(config1);
inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config4); inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config4);
@@ -13623,7 +13616,7 @@ public class ConnectivityServiceTest {
// Set MOBILE_DATA_PREFERRED_UIDS setting again with same uid as oem network preference. // Set MOBILE_DATA_PREFERRED_UIDS setting again with same uid as oem network preference.
final NativeUidRangeConfig config6 = new NativeUidRangeConfig(cellNetId, uidRanges3, final NativeUidRangeConfig config6 = new NativeUidRangeConfig(cellNetId, uidRanges3,
PREFERENCE_PRIORITY_MOBILE_DATA_PREFERERRED); PREFERENCE_ORDER_MOBILE_DATA_PREFERERRED);
setAndUpdateMobileDataPreferredUids(uids3); setAndUpdateMobileDataPreferredUids(uids3);
inorder.verify(mMockNetd, times(1)).networkRemoveUidRangesParcel(config3); inorder.verify(mMockNetd, times(1)).networkRemoveUidRangesParcel(config3);
inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config6); inorder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(config6);