Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9df3f48c authored by Yan Yan's avatar Yan Yan Committed by Gerrit Code Review
Browse files

Merge "Update #hashCode & #equals with mUnderlyingNetworkPriorities"

parents 761ad6ea d3a7de13

core/java/android/net/vcn/VcnCellUnderlyingNetworkPriority.java

+3 −3
Original line number Diff line number Diff line
@@ -151,7 +151,7 @@ public final class VcnCellUnderlyingNetworkPriority extends VcnUnderlyingNetwork 


    /** Retrieve the allowed PLMN IDs, or an empty set if any PLMN ID is acceptable. */

    @NonNull

    public Set<String> getAllowedPlmnIds() {

    public Set<String> getAllowedOperatorPlmnIds() {

        return Collections.unmodifiableSet(mAllowedNetworkPlmnIds);

    }
@@ -211,7 +211,7 @@ public final class VcnCellUnderlyingNetworkPriority extends VcnUnderlyingNetwork 
    }



    /** This class is used to incrementally build WifiNetworkPriority objects. */

    public static class Builder extends VcnUnderlyingNetworkPriority.Builder<Builder> {

    public static final class Builder extends VcnUnderlyingNetworkPriority.Builder<Builder> {

        @NonNull private final Set<String> mAllowedNetworkPlmnIds = new ArraySet<>();

        @NonNull private final Set<Integer> mAllowedSpecificCarrierIds = new ArraySet<>();
@@ -233,7 +233,7 @@ public final class VcnCellUnderlyingNetworkPriority extends VcnUnderlyingNetwork 
         *     and {@link SubscriptionInfo#getMncString()}.

         */

        @NonNull

        public Builder setAllowedPlmnIds(@NonNull Set<String> allowedNetworkPlmnIds) {

        public Builder setAllowedOperatorPlmnIds(@NonNull Set<String> allowedNetworkPlmnIds) {

            validatePlmnIds(allowedNetworkPlmnIds);



            mAllowedNetworkPlmnIds.clear();

core/java/android/net/vcn/VcnGatewayConnectionConfig.java

+2 −0
Original line number Diff line number Diff line
@@ -420,6 +420,7 @@ public final class VcnGatewayConnectionConfig { 
                mGatewayConnectionName,

                mTunnelConnectionParams,

                mExposedCapabilities,

                mUnderlyingNetworkPriorities,

                Arrays.hashCode(mRetryIntervalsMs),

                mMaxMtu);

    }
@@ -434,6 +435,7 @@ public final class VcnGatewayConnectionConfig { 
        return mGatewayConnectionName.equals(rhs.mGatewayConnectionName)

                && mTunnelConnectionParams.equals(rhs.mTunnelConnectionParams)

                && mExposedCapabilities.equals(rhs.mExposedCapabilities)

                && mUnderlyingNetworkPriorities.equals(rhs.mUnderlyingNetworkPriorities)

                && Arrays.equals(mRetryIntervalsMs, rhs.mRetryIntervalsMs)

                && mMaxMtu == rhs.mMaxMtu;

    }

core/java/android/net/vcn/VcnWifiUnderlyingNetworkPriority.java

+1 −1
Original line number Diff line number Diff line
@@ -79,7 +79,7 @@ public final class VcnWifiUnderlyingNetworkPriority extends VcnUnderlyingNetwork 
        }



        final VcnWifiUnderlyingNetworkPriority rhs = (VcnWifiUnderlyingNetworkPriority) other;

        return mSsid == rhs.mSsid;

        return mSsid.equals(rhs.mSsid);

    }



    /** @hide */

services/core/java/com/android/server/vcn/routeselection/NetworkPriorityClassifier.java

+2 −2
Original line number Diff line number Diff line
@@ -226,9 +226,9 @@ class NetworkPriorityClassifier { 
                        .getSystemService(TelephonyManager.class)

                        .createForSubscriptionId(subId);



        if (!networkPriority.getAllowedPlmnIds().isEmpty()) {

        if (!networkPriority.getAllowedOperatorPlmnIds().isEmpty()) {

            final String plmnId = subIdSpecificTelephonyMgr.getNetworkOperator();

            if (!networkPriority.getAllowedPlmnIds().contains(plmnId)) {

            if (!networkPriority.getAllowedOperatorPlmnIds().contains(plmnId)) {

                return false;

            }

        }

tests/vcn/java/android/net/vcn/VcnCellUnderlyingNetworkPriorityTest.java

+3 −3
Original line number Diff line number Diff line
@@ -36,7 +36,7 @@ public class VcnCellUnderlyingNetworkPriorityTest { 
        return new VcnCellUnderlyingNetworkPriority.Builder()

                .setNetworkQuality(NETWORK_QUALITY_OK)

                .setAllowMetered(true /* allowMetered */)

                .setAllowedPlmnIds(ALLOWED_PLMN_IDS)

                .setAllowedOperatorPlmnIds(ALLOWED_PLMN_IDS)

                .setAllowedSpecificCarrierIds(ALLOWED_CARRIER_IDS)

                .setAllowRoaming(true /* allowRoaming */)

                .setRequireOpportunistic(true /* requireOpportunistic */)
@@ -48,7 +48,7 @@ public class VcnCellUnderlyingNetworkPriorityTest { 
        final VcnCellUnderlyingNetworkPriority networkPriority = getTestNetworkPriority();

        assertEquals(NETWORK_QUALITY_OK, networkPriority.getNetworkQuality());

        assertTrue(networkPriority.allowMetered());

        assertEquals(ALLOWED_PLMN_IDS, networkPriority.getAllowedPlmnIds());

        assertEquals(ALLOWED_PLMN_IDS, networkPriority.getAllowedOperatorPlmnIds());

        assertEquals(ALLOWED_CARRIER_IDS, networkPriority.getAllowedSpecificCarrierIds());

        assertTrue(networkPriority.allowRoaming());

        assertTrue(networkPriority.requireOpportunistic());
@@ -60,7 +60,7 @@ public class VcnCellUnderlyingNetworkPriorityTest { 
                new VcnCellUnderlyingNetworkPriority.Builder().build();

        assertEquals(NETWORK_QUALITY_ANY, networkPriority.getNetworkQuality());

        assertFalse(networkPriority.allowMetered());

        assertEquals(new HashSet<String>(), networkPriority.getAllowedPlmnIds());

        assertEquals(new HashSet<String>(), networkPriority.getAllowedOperatorPlmnIds());

        assertEquals(new HashSet<Integer>(), networkPriority.getAllowedSpecificCarrierIds());

        assertFalse(networkPriority.allowRoaming());

        assertFalse(networkPriority.requireOpportunistic());