Modify the user default for "geotz enabled"

This commit modifies the default for geolocation-based time zone
detection from "whatever the user's location toggle is" to just always
be off. This seems the most likely launch state as the current plan of
record is to not opt-in upgrading users, and to explicitly set the
setting in the SetUp Wizard for new users.

Bug: 151304765
Test: Build / TH only
Change-Id: Ifa5914b4d99708b54907259c50bd455897437225
This commit is contained in:
Neil Fuller
2020-10-12 14:04:39 +01:00
parent 687d46aa30
commit 93f62ab79a

View File

@@ -199,10 +199,10 @@ public final class TimeZoneDetectorCallbackImpl implements TimeZoneDetectorStrat
}
private boolean isGeoDetectionEnabled(@UserIdInt int userId) {
final boolean locationEnabled = isLocationEnabled(userId);
final boolean geoDetectionEnabledByDefault = false;
return Settings.Secure.getIntForUser(mCr,
Settings.Secure.LOCATION_TIME_ZONE_DETECTION_ENABLED,
locationEnabled ? 1 : 0 /* defaultValue */, userId) != 0;
(geoDetectionEnabledByDefault ? 1 : 0) /* defaultValue */, userId) != 0;
}
private void setGeoDetectionEnabled(@UserIdInt int userId, boolean enabled) {