Convert DeviceInfoSettings into a dashboard fragment.
This is needed to use logging from DashboardFragemnt.
This page is now a mixture of PrefernceController and non-controller
preferences.
Also permanently removed system update preference from xml to simplify
migration
Todo: convert the rest of preference into controller.
Bug: 34774945
Test: make RunSettingsRoboTests
Change-Id: Ie5130ea7377db2ccf2236cdf48e5cc26d1347d7a
diff --git a/src/com/android/settings/DeviceInfoSettings.java b/src/com/android/settings/DeviceInfoSettings.java
index 0da44cd..bd07685 100644
--- a/src/com/android/settings/DeviceInfoSettings.java
+++ b/src/com/android/settings/DeviceInfoSettings.java
@@ -28,18 +28,19 @@
import android.os.UserManager;
import android.provider.SearchIndexableResource;
import android.provider.Settings;
-import android.support.annotation.VisibleForTesting;
import android.support.v7.preference.Preference;
import android.support.v7.preference.PreferenceGroup;
import android.text.TextUtils;
import android.util.Log;
import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
+import com.android.settings.core.PreferenceController;
+import com.android.settings.dashboard.DashboardFragment;
import com.android.settings.dashboard.SummaryLoader;
import com.android.settings.deviceinfo.AdditionalSystemUpdatePreferenceController;
import com.android.settings.deviceinfo.BuildNumberPreferenceController;
+import com.android.settings.deviceinfo.ManualPreferenceController;
import com.android.settings.deviceinfo.SystemUpdatePreferenceController;
-import com.android.settings.overlay.FeatureFactory;
import com.android.settings.search.BaseSearchIndexProvider;
import com.android.settings.search.Indexable;
import com.android.settingslib.DeviceInfoUtils;
@@ -51,13 +52,11 @@
import static com.android.settingslib.RestrictedLockUtils.EnforcedAdmin;
-public class DeviceInfoSettings extends SettingsPreferenceFragment implements Indexable {
+public class DeviceInfoSettings extends DashboardFragment implements Indexable {
private static final String LOG_TAG = "DeviceInfoSettings";
- private static final String KEY_MANUAL = "manual";
private static final String KEY_REGULATORY_INFO = "regulatory_info";
- private static final String KEY_SYSTEM_UPDATE_SETTINGS = "system_update_settings";
private static final String PROPERTY_URL_SAFETYLEGAL = "ro.url.safetylegal";
private static final String PROPERTY_SELINUX_STATUS = "ro.build.selinux";
private static final String KEY_KERNEL_VERSION = "kernel_version";
@@ -73,8 +72,6 @@
long[] mHits = new long[3];
- private SystemUpdatePreferenceController mSystemUpdatePreferenceController;
- private AdditionalSystemUpdatePreferenceController mAdditionalSystemUpdatePreferenceController;
private BuildNumberPreferenceController mBuildNumberPreferenceController;
private UserManager mUm;
@@ -93,6 +90,12 @@
}
@Override
+ public void onAttach(Context context) {
+ super.onAttach(context);
+ mUm = (UserManager) context.getSystemService(Context.USER_SERVICE);
+ }
+
+ @Override
public void onActivityResult(int requestCode, int resultCode, Intent data) {
if (mBuildNumberPreferenceController.onActivityResult(requestCode, resultCode, data)) {
return;
@@ -103,14 +106,6 @@
@Override
public void onCreate(Bundle icicle) {
super.onCreate(icicle);
- final Activity activity = getActivity();
- mUm = UserManager.get(activity);
- mAdditionalSystemUpdatePreferenceController =
- new AdditionalSystemUpdatePreferenceController(activity);
- mBuildNumberPreferenceController =
- new BuildNumberPreferenceController(activity, activity, this /* fragment */);
- getLifecycle().addObserver(mBuildNumberPreferenceController);
- addPreferencesFromResource(R.xml.device_info_settings);
setStringSummary(KEY_FIRMWARE_VERSION, Build.VERSION.RELEASE);
findPreference(KEY_FIRMWARE_VERSION).setEnabled(true);
@@ -125,7 +120,6 @@
setValueSummary(KEY_BASEBAND_VERSION, "gsm.version.baseband");
setStringSummary(KEY_DEVICE_MODEL, Build.MODEL + DeviceInfoUtils.getMsvSuffix());
setValueSummary(KEY_EQUIPMENT_ID, PROPERTY_EQUIPMENT_ID);
- mBuildNumberPreferenceController.displayPreference(getPreferenceScreen());
findPreference(KEY_KERNEL_VERSION).setSummary(DeviceInfoUtils.getFormattedKernelVersion());
if (!SELinux.isSELinuxEnabled()) {
@@ -158,16 +152,6 @@
getPreferenceScreen().removePreference(findPreference(KEY_DEVICE_FEEDBACK));
}
- /*
- * Settings is a generic app and should not contain any device-specific
- * info.
- */
- displaySystemUpdates(activity);
- mAdditionalSystemUpdatePreferenceController.displayPreference(getPreferenceScreen());
-
- // Remove manual entry if none present.
- removePreferenceIfBoolFalse(KEY_MANUAL, R.bool.config_show_manual);
-
// Remove regulatory labels if no activity present to handle intent.
removePreferenceIfActivityMissing(
KEY_REGULATORY_INFO, Settings.ACTION_SHOW_REGULATORY_INFO);
@@ -222,25 +206,33 @@
} else if (preference.getKey().equals(KEY_DEVICE_FEEDBACK)) {
sendFeedback();
}
- if (mSystemUpdatePreferenceController != null) {
- mSystemUpdatePreferenceController.handlePreferenceTreeClick(preference);
- }
return super.onPreferenceTreeClick(preference);
}
- @VisibleForTesting
- void displaySystemUpdates(Context context) {
- if (!FeatureFactory.getFactory(context).getDashboardFeatureProvider(context).isEnabled()) {
- mSystemUpdatePreferenceController
- = new SystemUpdatePreferenceController(context, UserManager.get(context));
- mSystemUpdatePreferenceController.displayPreference(getPreferenceScreen());
- } else {
- getPreferenceScreen().removePreference(findPreference(KEY_SYSTEM_UPDATE_SETTINGS));
- }
+ @Override
+ protected String getLogTag() {
+ return LOG_TAG;
+ }
+
+ @Override
+ protected int getPreferenceScreenResId() {
+ return R.xml.device_info_settings;
+ }
+
+ @Override
+ protected List<PreferenceController> getPreferenceControllers(Context context) {
+ final List<PreferenceController> controllers = new ArrayList<>();
+ mBuildNumberPreferenceController =
+ new BuildNumberPreferenceController(context, getActivity(), this /* fragment */);
+ getLifecycle().addObserver(mBuildNumberPreferenceController);
+ controllers.add(mBuildNumberPreferenceController);
+ controllers.add(new AdditionalSystemUpdatePreferenceController(context));
+ controllers.add(new ManualPreferenceController(context));
+ return controllers;
}
private void removePreferenceIfPropertyMissing(PreferenceGroup preferenceGroup,
- String preference, String property ) {
+ String preference, String property) {
if (SystemProperties.get(property).equals("")) {
// Property is missing so remove preference from group
try {
@@ -262,15 +254,6 @@
}
}
- private void removePreferenceIfBoolFalse(String preference, int resId) {
- if (!getResources().getBoolean(resId)) {
- Preference pref = findPreference(preference);
- if (pref != null) {
- getPreferenceScreen().removePreference(pref);
- }
- }
- }
-
private void setStringSummary(String preference, String value) {
try {
findPreference(preference).setSummary(value);
diff --git a/src/com/android/settings/deviceinfo/ManualPreferenceController.java b/src/com/android/settings/deviceinfo/ManualPreferenceController.java
new file mode 100644
index 0000000..dbaa00f
--- /dev/null
+++ b/src/com/android/settings/deviceinfo/ManualPreferenceController.java
@@ -0,0 +1,41 @@
+/*
+ * Copyright (C) 2017 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.android.settings.deviceinfo;
+
+import android.content.Context;
+
+import com.android.settings.R;
+import com.android.settings.core.PreferenceController;
+
+public class ManualPreferenceController extends PreferenceController {
+
+ private static final String KEY_MANUAL = "manual";
+
+ public ManualPreferenceController(Context context) {
+ super(context);
+ }
+
+ @Override
+ public boolean isAvailable() {
+ return mContext.getResources().getBoolean(R.bool.config_show_manual);
+ }
+
+ @Override
+ public String getPreferenceKey() {
+ return KEY_MANUAL;
+ }
+}