From f409181f4de2e067018d4375e5a0cb020e7740b1 Mon Sep 17 00:00:00 2001 From: Tom O'Neill Date: Tue, 19 May 2015 10:32:46 -0700 Subject: [PATCH] LocationSettings javadoc describes what's in the UI - In particular, tells OEMs how to add their own settings Change-Id: I45e4fb7ff59705a2bcb53a72c71fd230f2e085c5 # Conflicts: # src/com/android/settings/location/LocationSettings.java --- .../settings/location/LocationSettings.java | 25 +++++++++++++++++-- 1 file changed, 23 insertions(+), 2 deletions(-) diff --git a/src/com/android/settings/location/LocationSettings.java b/src/com/android/settings/location/LocationSettings.java index ba5cd924810..b36d0ad7302 100644 --- a/src/com/android/settings/location/LocationSettings.java +++ b/src/com/android/settings/location/LocationSettings.java @@ -33,7 +33,6 @@ import android.preference.SwitchPreference; import android.provider.Settings; import android.util.Log; import android.widget.Switch; - import com.android.settings.R; import com.android.settings.SettingsActivity; import com.android.settings.Utils; @@ -44,7 +43,29 @@ import java.util.Comparator; import java.util.List; /** - * Location access settings. + * System location settings (Settings > Location). The screen has three parts: + * + *

+ * Note that as of KitKat, the {@link SettingInjectorService} is the preferred method for OEMs to + * add their own settings to this page, rather than directly modifying the framework code. Among + * other things, this simplifies integration with future changes to the default (AOSP) + * implementation. */ public class LocationSettings extends LocationSettingsBase implements SwitchBar.OnSwitchChangeListener {