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

Commit 7738b847 authored by Chalard Jean's avatar Chalard Jean
Browse files

[NS B07] More simplification 

The new reassignment does not contain these useless lines
any more.

Test: FrameworksNetTests
Change-Id: I1583aebe94e529ce2b36e191a6e1f49c976bf29a
parent cdd4c450

services/core/java/com/android/server/ConnectivityService.java

+0 −6
Original line number Diff line number Diff line
@@ -6709,10 +6709,6 @@ public class ConnectivityService extends IConnectivityManager.Stub 
        // the linger status.

        for (final NetworkReassignment.RequestReassignment event :

                changes.getRequestReassignments()) {

            // The rematch is seeded with an entry for each request, and requests that don't

            // change satisfiers have the same network as old and new.

            // TODO : remove these entries when they are not needed any more.

            if (event.mOldNetwork == event.mNewNetwork) continue;

            updateSatisfiersForRematchRequest(event.mRequest, event.mOldNetwork,

                    event.mNewNetwork, now);

        }
@@ -6741,8 +6737,6 @@ public class ConnectivityService extends IConnectivityManager.Stub 
        // before LegacyTypeTracker sends legacy broadcasts

        for (final NetworkReassignment.RequestReassignment event :

                changes.getRequestReassignments()) {

            if (event.mOldNetwork == event.mNewNetwork) continue;



            // Tell NetworkProviders about the new score, so they can stop

            // trying to connect if they know they cannot match it.

            // TODO - this could get expensive if there are a lot of outstanding requests for this