Resolve API council feedback
Test: built and verified logs Fix: 225116248 Ignore-AOSP-First: nearby_not_in_aosp_yet Change-Id: Ic30aee650fd41287d3a363491fa602cd990467a8
This commit is contained in:
parent
acff90b51f
commit
1b2a5bf724
4 changed files with 6 additions and 7 deletions
|
|
@ -301,7 +301,7 @@ package android.nearby {
|
|||
}
|
||||
|
||||
public class NearbyManager {
|
||||
method public static boolean getFastPairScanEnabled(@NonNull android.content.Context, boolean);
|
||||
method public static boolean getFastPairScanEnabled(@NonNull android.content.Context);
|
||||
method public static void setFastPairScanEnabled(@NonNull android.content.Context, boolean);
|
||||
method @RequiresPermission(allOf={android.Manifest.permission.BLUETOOTH_ADVERTISE, android.Manifest.permission.BLUETOOTH_PRIVILEGED}) public void startBroadcast(@NonNull android.nearby.BroadcastRequest, @NonNull java.util.concurrent.Executor, @NonNull android.nearby.BroadcastCallback);
|
||||
method @RequiresPermission(allOf={android.Manifest.permission.BLUETOOTH_SCAN, android.Manifest.permission.BLUETOOTH_PRIVILEGED}) public int startScan(@NonNull android.nearby.ScanRequest, @NonNull java.util.concurrent.Executor, @NonNull android.nearby.ScanCallback);
|
||||
|
|
|
|||
|
|
@ -244,12 +244,11 @@ public class NearbyManager {
|
|||
* Read from {@link Settings} whether Fast Pair scan is enabled.
|
||||
*
|
||||
* @param context the {@link Context} to query the setting
|
||||
* @param def the default value if no setting value
|
||||
* @return whether the Fast Pair is enabled
|
||||
*/
|
||||
public static boolean getFastPairScanEnabled(@NonNull Context context, boolean def) {
|
||||
public static boolean getFastPairScanEnabled(@NonNull Context context) {
|
||||
final int enabled = Settings.Secure.getInt(
|
||||
context.getContentResolver(), FAST_PAIR_SCAN_ENABLED, (def ? 1 : 0));
|
||||
context.getContentResolver(), FAST_PAIR_SCAN_ENABLED, 0);
|
||||
return enabled != 0;
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -155,7 +155,7 @@ public class FastPairManager {
|
|||
|
||||
Locator.getFromContextWrapper(mLocatorContextWrapper, FastPairCacheManager.class);
|
||||
// Default false for now.
|
||||
mScanEnabled = NearbyManager.getFastPairScanEnabled(mLocatorContextWrapper, false);
|
||||
mScanEnabled = NearbyManager.getFastPairScanEnabled(mLocatorContextWrapper.getContext());
|
||||
registerFastPairScanChangeContentObserver(mLocatorContextWrapper.getContentResolver());
|
||||
}
|
||||
|
||||
|
|
@ -357,7 +357,7 @@ public class FastPairManager {
|
|||
public void onChange(boolean selfChange, Uri uri) {
|
||||
super.onChange(selfChange, uri);
|
||||
setScanEnabled(
|
||||
NearbyManager.getFastPairScanEnabled(mLocatorContextWrapper, mScanEnabled));
|
||||
NearbyManager.getFastPairScanEnabled(mLocatorContextWrapper.getContext()));
|
||||
}
|
||||
};
|
||||
try {
|
||||
|
|
|
|||
|
|
@ -140,7 +140,7 @@ public class NearbyManagerTest {
|
|||
@SdkSuppress(minSdkVersion = 32, codeName = "T")
|
||||
public void testSettingsEnable() {
|
||||
NearbyManager.setFastPairScanEnabled(mContext, false);
|
||||
assertThat(NearbyManager.getFastPairScanEnabled(mContext, true)).isFalse();
|
||||
assertThat(NearbyManager.getFastPairScanEnabled(mContext)).isFalse();
|
||||
}
|
||||
|
||||
private void enableBluetooth() {
|
||||
|
|
|
|||
Loading…
Reference in a new issue