Adapt new EthernetManager APIs in Settings.
EthernetManager is going to be moved to Connectivity mainline module and new EthernetManager management APIs have been exposed. This CL adapts new APIs in the settings in advance, the changes include: 1. use addInterfaceStateListener and removeInterfaceStateListener. 2. rely on the onInterfaceStateChanged callback to receive the Ethernet interface state update, to replace the getAvailableInterfaces and getConfiguration. 3. after the Ethernet mainline migration completes, Settings cannot access the platform resource such as config_ethernet_iface_regex, instead, check the availability of Ethernet interface by checking if either any of FEATURE_ETHERNET and FEATURE_USB_HOST is supported. Bug: 210586283 Bug: 218798003 Test: m Test: manually verify that device can access the Internet via Ethernet Test: manually verify that device can share the Internet via Ethernet tethering Test: make RunSettingsRoboTests ROBOTEST_FILTER=<modified test cases> EthernetTetherPreferenceControllerTest AllInOneTetherSettingsTest Change-Id: I9e10481e1751975772a24db29568aa26bb85cd70
This commit is contained in:
@@ -21,52 +21,63 @@ import android.net.EthernetManager;
|
||||
import android.net.TetheringManager;
|
||||
import android.os.Handler;
|
||||
import android.os.Looper;
|
||||
import android.text.TextUtils;
|
||||
|
||||
import androidx.lifecycle.Lifecycle;
|
||||
import androidx.lifecycle.OnLifecycleEvent;
|
||||
|
||||
import com.android.internal.annotations.VisibleForTesting;
|
||||
|
||||
import java.util.HashSet;
|
||||
|
||||
/**
|
||||
* This controller helps to manage the switch state and visibility of ethernet tether switch
|
||||
* preference.
|
||||
*/
|
||||
public final class EthernetTetherPreferenceController extends TetherBasePreferenceController {
|
||||
|
||||
private final String mEthernetRegex;
|
||||
private final HashSet<String> mAvailableInterfaces = new HashSet<>();
|
||||
private final EthernetManager mEthernetManager;
|
||||
|
||||
@VisibleForTesting
|
||||
EthernetManager.Listener mEthernetListener;
|
||||
EthernetManager.InterfaceStateListener mEthernetListener;
|
||||
|
||||
public EthernetTetherPreferenceController(Context context, String preferenceKey) {
|
||||
super(context, preferenceKey);
|
||||
mEthernetRegex = context.getString(
|
||||
com.android.internal.R.string.config_ethernet_iface_regex);
|
||||
mEthernetManager = (EthernetManager) context.getSystemService(Context.ETHERNET_SERVICE);
|
||||
mEthernetManager = context.getSystemService(EthernetManager.class);
|
||||
}
|
||||
|
||||
@OnLifecycleEvent(Lifecycle.Event.ON_START)
|
||||
public void onStart() {
|
||||
mEthernetListener = (iface, isAvailable) -> updateState(mPreference);
|
||||
mEthernetListener = (iface, state, role, configuration) -> {
|
||||
if (state == EthernetManager.STATE_LINK_UP) {
|
||||
mAvailableInterfaces.add(iface);
|
||||
} else {
|
||||
mAvailableInterfaces.remove(iface);
|
||||
}
|
||||
updateState(mPreference);
|
||||
};
|
||||
final Handler handler = new Handler(Looper.getMainLooper());
|
||||
// Executor will execute to post the updateState event to a new handler which is created
|
||||
// from the main looper when the {@link EthernetManager.Listener.onAvailabilityChanged}
|
||||
// is triggerd.
|
||||
mEthernetManager.addListener(mEthernetListener, r -> handler.post(r));
|
||||
if (mEthernetManager != null) {
|
||||
mEthernetManager.addInterfaceStateListener(r -> handler.post(r), mEthernetListener);
|
||||
}
|
||||
}
|
||||
|
||||
@OnLifecycleEvent(Lifecycle.Event.ON_STOP)
|
||||
public void onStop() {
|
||||
mEthernetManager.removeListener(mEthernetListener);
|
||||
mEthernetListener = null;
|
||||
if (mEthernetManager != null) {
|
||||
mEthernetManager.removeInterfaceStateListener(mEthernetListener);
|
||||
}
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean shouldEnable() {
|
||||
ensureRunningOnMainLoopThread();
|
||||
String[] available = mTm.getTetherableIfaces();
|
||||
for (String s : available) {
|
||||
if (s.matches(mEthernetRegex)) {
|
||||
if (mAvailableInterfaces.contains(s)) {
|
||||
return true;
|
||||
}
|
||||
}
|
||||
@@ -75,11 +86,19 @@ public final class EthernetTetherPreferenceController extends TetherBasePreferen
|
||||
|
||||
@Override
|
||||
public boolean shouldShow() {
|
||||
return !TextUtils.isEmpty(mEthernetRegex);
|
||||
return mEthernetManager != null;
|
||||
}
|
||||
|
||||
@Override
|
||||
public int getTetherType() {
|
||||
return TetheringManager.TETHERING_ETHERNET;
|
||||
}
|
||||
|
||||
private void ensureRunningOnMainLoopThread() {
|
||||
if (Looper.getMainLooper().getThread() != Thread.currentThread()) {
|
||||
throw new IllegalStateException(
|
||||
"Not running on main loop thread: "
|
||||
+ Thread.currentThread().getName());
|
||||
}
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user