From dff5df5b650d63e6f65fe5fc20a32ad32a6e3143 Mon Sep 17 00:00:00 2001
From: Motomu Utsumi <motomuman@google.com>
Date: Tue, 14 May 2024 18:19:43 +0900
Subject: [PATCH] Rename closePendingFrozenSockets to destroyPendingSockets

As a preparation to update CS to destroy sockets for background apps,
renamed socket destroy methods to non specific to frozen apps

Bug: 300681644
Test: TH
Change-Id: I64c108aee1ee0d60df30ef055c8c9df8c6562d0f
---
 .../com/android/server/ConnectivityService.java  | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/service/src/com/android/server/ConnectivityService.java b/service/src/com/android/server/ConnectivityService.java
index 82d6fb9f24..92efe2b5a6 100755
--- a/service/src/com/android/server/ConnectivityService.java
+++ b/service/src/com/android/server/ConnectivityService.java
@@ -3378,13 +3378,13 @@ public class ConnectivityService extends IConnectivityManager.Stub
         }
     }
 
-    private void closePendingFrozenSockets() {
+    private void destroyPendingSockets() {
         ensureRunningOnConnectivityServiceThread();
 
         try {
             mDeps.destroyLiveTcpSocketsByOwnerUids(mPendingFrozenUids);
         } catch (SocketException | InterruptedIOException | ErrnoException e) {
-            loge("Failed to close pending frozen app sockets: " + e);
+            loge("Failed to destroy sockets: " + e);
         }
         mPendingFrozenUids.clear();
     }
@@ -3407,17 +3407,17 @@ public class ConnectivityService extends IConnectivityManager.Stub
                 && params.isActive
                 && isCellNetworkActivity
                 && !mPendingFrozenUids.isEmpty()) {
-            closePendingFrozenSockets();
+            destroyPendingSockets();
         }
     }
 
     /**
-     * If the cellular network is no longer the default network, close pending frozen sockets.
+     * If the cellular network is no longer the default network, destroy pending sockets.
      *
      * @param newNetwork new default network
      * @param oldNetwork old default network
      */
-    private void maybeClosePendingFrozenSockets(NetworkAgentInfo newNetwork,
+    private void maybeDestroyPendingSockets(NetworkAgentInfo newNetwork,
             NetworkAgentInfo oldNetwork) {
         final boolean isOldNetworkCellular = oldNetwork != null
                 && oldNetwork.networkCapabilities.hasTransport(TRANSPORT_CELLULAR);
@@ -3427,7 +3427,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
         if (isOldNetworkCellular
                 && !isNewNetworkCellular
                 && !mPendingFrozenUids.isEmpty()) {
-            closePendingFrozenSockets();
+            destroyPendingSockets();
         }
     }
 
@@ -5208,7 +5208,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
 
                 if (mDefaultRequest == nri) {
                     mNetworkActivityTracker.updateDefaultNetwork(null /* newNetwork */, nai);
-                    maybeClosePendingFrozenSockets(null /* newNetwork */, nai);
+                    maybeDestroyPendingSockets(null /* newNetwork */, nai);
                     ensureNetworkTransitionWakelock(nai.toShortString());
                 }
             }
@@ -10084,7 +10084,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
             mLingerMonitor.noteLingerDefaultNetwork(oldDefaultNetwork, newDefaultNetwork);
         }
         mNetworkActivityTracker.updateDefaultNetwork(newDefaultNetwork, oldDefaultNetwork);
-        maybeClosePendingFrozenSockets(newDefaultNetwork, oldDefaultNetwork);
+        maybeDestroyPendingSockets(newDefaultNetwork, oldDefaultNetwork);
         mProxyTracker.setDefaultProxy(null != newDefaultNetwork
                 ? newDefaultNetwork.linkProperties.getHttpProxy() : null);
         resetHttpProxyForNonDefaultNetwork(oldDefaultNetwork);