commit b02eb1ccfbe059766320fc320fd91e2c80717eb9
author David Su <dysu@google.com> Wed Jul 15 21:51:29 2020 -0700
committer David Su <dysu@google.com> Thu Jul 16 18:24:51 2020 -0700
tree 7ef6a0e569f599776f1789556dc8bd11b2838c74
parent 0b0ba8b57000b7b735b856dcaf23e61f2d48880f

ClientModeManager: remove unused syncGetWifiStateByName

This method is unused by callers of ClientModeManager,
it is only used by callers of ClientModeImpl.

Bug: 159060934
Test: atest FrameworksWifiTests
Change-Id: Ibc0568f3380c79f4ece7831a9d6f0f54460e6f77

service/java/com/android/server/wifi/ClientModeImpl.java

diff --git a/service/java/com/android/server/wifi/ClientModeImpl.java b/service/java/com/android/server/wifi/ClientModeImpl.java
index ff12ae7..0e55449 100644
--- a/service/java/com/android/server/wifi/ClientModeImpl.java
+++ b/service/java/com/android/server/wifi/ClientModeImpl.java
@@ -1361,7 +1361,8 @@
     /**
      * Converts the current wifi state to a printable form.
      */
-    public String syncGetWifiStateByName() {
+    @VisibleForTesting
+    String syncGetWifiStateByName() {
         switch (mWifiState.get()) {
             case WIFI_STATE_DISABLING:
                 return "disabling";

service/java/com/android/server/wifi/ClientModeManager.java

diff --git a/service/java/com/android/server/wifi/ClientModeManager.java b/service/java/com/android/server/wifi/ClientModeManager.java
index 3b84e6f..e7b1d70 100644
--- a/service/java/com/android/server/wifi/ClientModeManager.java
+++ b/service/java/com/android/server/wifi/ClientModeManager.java
@@ -73,8 +73,6 @@
 
     DhcpResultsParcelable syncGetDhcpResultsParcelable();
 
-    String syncGetWifiStateByName();
-
     long syncGetSupportedFeatures();
 
     boolean syncStartSubscriptionProvisioning(int callingUid, OsuProvider provider,

service/java/com/android/server/wifi/ConcreteClientModeManager.java

diff --git a/service/java/com/android/server/wifi/ConcreteClientModeManager.java b/service/java/com/android/server/wifi/ConcreteClientModeManager.java
index 658f4a0..f70be35 100644
--- a/service/java/com/android/server/wifi/ConcreteClientModeManager.java
+++ b/service/java/com/android/server/wifi/ConcreteClientModeManager.java
@@ -804,11 +804,6 @@
     }
 
     @Override
-    public String syncGetWifiStateByName() {
-        return mClientModeImpl.syncGetWifiStateByName();
-    }
-
-    @Override
     public long syncGetSupportedFeatures() {
         return mClientModeImpl.syncGetSupportedFeatures();
     }

service/java/com/android/server/wifi/DefaultClientModeManager.java

diff --git a/service/java/com/android/server/wifi/DefaultClientModeManager.java b/service/java/com/android/server/wifi/DefaultClientModeManager.java
index cdf4bd1..6932f99 100644
--- a/service/java/com/android/server/wifi/DefaultClientModeManager.java
+++ b/service/java/com/android/server/wifi/DefaultClientModeManager.java
@@ -134,11 +134,6 @@
         return new DhcpResultsParcelable();
     }
 
-    @Override
-    public String syncGetWifiStateByName() {
-        return null;
-    }
-
     public long syncGetSupportedFeatures() {
         return 0L;
     }