Add a little comment about the enabled value
Change-Id: I4ac0b864f55992242b6a3b0d8ffb328f23f6b645
diff --git a/location/java/android/location/SettingInjectorService.java b/location/java/android/location/SettingInjectorService.java
index 4bfa150..dbc3f27 100644
--- a/location/java/android/location/SettingInjectorService.java
+++ b/location/java/android/location/SettingInjectorService.java
@@ -176,8 +176,15 @@
/**
* Constructor.
+ * <p/>
+ * Note that to prevent churn in the settings list, there is no support for dynamically
+ * choosing to hide a setting. Instead you should provide a {@code enabled} value of false,
+ * which will gray the setting out and disable the link from "Settings > Location"
+ * to your setting activity. One reason why you might choose to do this is if
+ * {@link android.provider.Settings.Secure#getLocationMode(android.content.ContentResolver)}
+ * is {@link android.provider.Settings.Secure#LOCATION_MODE_OFF}.
*
- * @param summary the {@link Preference#getSummary()} value
+ * @param summary the {@link Preference#getSummary()} value (allowed to be null or empty)
* @param enabled the {@link Preference#isEnabled()} value
*/
public Status(String summary, boolean enabled) {