joy/android_packages_modules_Connectivity
Watch
1
Fork
You've already forked android_packages_modules_Connectivity
 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge "Force TETHER_ENABLE_SYNC_SM to be enabled by default on 25Q2+" into main am: 811db343cd

Browse source 
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/3541181

Change-Id: I405e408729e822ca181251e4e37b16ad638df411
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Treehugger Robot 2025-03-19 00:17:14 -07:00 • committed by Automerger Merge Worker
commit 553e1884d4
3 changed files with 16 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

3
Tethering/src/com/android/networkstack/tethering/TetheringConfiguration.java
View file

@ -24,6 +24,7 @@ import static android.net.ConnectivityManager.TYPE_MOBILE_HIPRI;
import static android.provider.DeviceConfig.NAMESPACE_CONNECTIVITY;
import static com.android.networkstack.apishim.ConstantsShim.KEY_CARRIER_SUPPORTS_TETHERING_BOOL;
import static com.android.net.module.util.SdkUtil.isAtLeast25Q2;
import android.content.ContentResolver;
import android.content.Context;
@ -395,7 +396,7 @@ public class TetheringConfiguration {
* use the async state machine.
*/
public void readEnableSyncSM(final Context ctx) {
USE_SYNC_SM = mDeps.isFeatureNotChickenedOut(ctx, TETHER_ENABLE_SYNC_SM);
USE_SYNC_SM = isAtLeast25Q2() || mDeps.isFeatureNotChickenedOut(ctx, TETHER_ENABLE_SYNC_SM);
}
/** Does the dumping.*/

8
Tethering/tests/unit/src/com/android/networkstack/tethering/TetheringConfigurationTest.java
View file

@ -63,6 +63,7 @@ import com.android.internal.util.test.BroadcastInterceptingContext;
import com.android.internal.util.test.FakeSettingsProvider;
import com.android.modules.utils.build.SdkLevel;
import com.android.net.module.util.DeviceConfigUtils;
import com.android.net.module.util.SdkUtil;
import com.android.net.module.util.SharedLog;
import com.android.testutils.DevSdkIgnoreRule;
import com.android.testutils.DevSdkIgnoreRule.IgnoreAfter;
@ -779,7 +780,10 @@ public class TetheringConfigurationTest {
setTetherEnableSyncSMFlagEnabled(true);
assertEnableSyncSM(true);
setTetherEnableSyncSMFlagEnabled(false);
assertEnableSyncSM(false);
// Feature is enabled by default after 25Q2 release.
if (!SdkUtil.isAtLeast25Q2()) {
setTetherEnableSyncSMFlagEnabled(false);
assertEnableSyncSM(false);
}
}
}

8
staticlibs/framework/com/android/net/module/util/SdkUtil.java
View file

@ -16,7 +16,10 @@
package com.android.net.module.util;
import static android.os.Build.VERSION.SDK_INT;
import android.annotation.Nullable;
import android.os.Build;
/**
* Utilities to deal with multiple SDKs in a single mainline module.
@ -46,4 +49,9 @@ public class SdkUtil {
this.value = value;
}
}
/** Checks if the device is running on a release version of Android Baklava or newer */
public static boolean isAtLeast25Q2() {
return SDK_INT >= 36 || (SDK_INT == 35 && "Baklava".equals(Build.VERSION.CODENAME));
}
}