[Settings] Code refactor for data usage
Code refactor Bug: 151751844 Test: make RunSettingsRoboTests -j ROBOTEST_FILTER=DataUsageSummaryPreferenceControllerTest Change-Id: I22b323bae0b18cca26c62d047fdb504c012518b5
This commit is contained in:
@@ -64,7 +64,7 @@ public class DataUsageSummaryPreferenceController extends TelephonyBasePreferenc
|
|||||||
private static final float RELATIVE_SIZE_LARGE = 1.25f * 1.25f; // (1/0.8)^2
|
private static final float RELATIVE_SIZE_LARGE = 1.25f * 1.25f; // (1/0.8)^2
|
||||||
private static final float RELATIVE_SIZE_SMALL = 1.0f / RELATIVE_SIZE_LARGE; // 0.8^2
|
private static final float RELATIVE_SIZE_SMALL = 1.0f / RELATIVE_SIZE_LARGE; // 0.8^2
|
||||||
|
|
||||||
private final EntityHeaderController mEntityHeaderController;
|
private EntityHeaderController mEntityHeaderController;
|
||||||
private final Lifecycle mLifecycle;
|
private final Lifecycle mLifecycle;
|
||||||
private final PreferenceFragmentCompat mFragment;
|
private final PreferenceFragmentCompat mFragment;
|
||||||
protected DataUsageController mDataUsageController;
|
protected DataUsageController mDataUsageController;
|
||||||
@@ -103,8 +103,6 @@ public class DataUsageSummaryPreferenceController extends TelephonyBasePreferenc
|
|||||||
Lifecycle lifecycle, PreferenceFragmentCompat fragment, int subscriptionId) {
|
Lifecycle lifecycle, PreferenceFragmentCompat fragment, int subscriptionId) {
|
||||||
super(activity, KEY);
|
super(activity, KEY);
|
||||||
|
|
||||||
mEntityHeaderController = EntityHeaderController.newInstance(activity,
|
|
||||||
fragment, null);
|
|
||||||
mLifecycle = lifecycle;
|
mLifecycle = lifecycle;
|
||||||
mFragment = fragment;
|
mFragment = fragment;
|
||||||
init(subscriptionId);
|
init(subscriptionId);
|
||||||
@@ -165,6 +163,10 @@ public class DataUsageSummaryPreferenceController extends TelephonyBasePreferenc
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
public void onStart() {
|
public void onStart() {
|
||||||
|
if (mEntityHeaderController == null) {
|
||||||
|
mEntityHeaderController =
|
||||||
|
EntityHeaderController.newInstance((Activity) mContext, mFragment, null);
|
||||||
|
}
|
||||||
RecyclerView view = mFragment.getListView();
|
RecyclerView view = mFragment.getListView();
|
||||||
mEntityHeaderController.setRecyclerView(view, mLifecycle);
|
mEntityHeaderController.setRecyclerView(view, mLifecycle);
|
||||||
mEntityHeaderController.styleActionBar((Activity) mContext);
|
mEntityHeaderController.styleActionBar((Activity) mContext);
|
||||||
@@ -197,16 +199,20 @@ public class DataUsageSummaryPreferenceController extends TelephonyBasePreferenc
|
|||||||
public void updateState(Preference preference) {
|
public void updateState(Preference preference) {
|
||||||
DataUsageSummaryPreference summaryPreference = (DataUsageSummaryPreference) preference;
|
DataUsageSummaryPreference summaryPreference = (DataUsageSummaryPreference) preference;
|
||||||
|
|
||||||
final DataUsageController.DataUsageInfo info;
|
final boolean isSimCardAdded = hasSim();
|
||||||
|
if (!isSimCardAdded) {
|
||||||
|
mDefaultTemplate = NetworkTemplate.buildTemplateWifiWildcard();
|
||||||
|
}
|
||||||
|
|
||||||
|
final DataUsageController.DataUsageInfo info =
|
||||||
|
mDataUsageController.getDataUsageInfo(mDefaultTemplate);
|
||||||
|
|
||||||
final SubscriptionInfo subInfo = getSubscriptionInfo(mSubId);
|
final SubscriptionInfo subInfo = getSubscriptionInfo(mSubId);
|
||||||
if (hasSim()) {
|
if (isSimCardAdded) {
|
||||||
info = mDataUsageController.getDataUsageInfo(mDefaultTemplate);
|
|
||||||
mDataInfoController.updateDataLimit(info, mPolicyEditor.getPolicy(mDefaultTemplate));
|
mDataInfoController.updateDataLimit(info, mPolicyEditor.getPolicy(mDefaultTemplate));
|
||||||
summaryPreference.setWifiMode(/* isWifiMode */ false,
|
summaryPreference.setWifiMode(/* isWifiMode */ false,
|
||||||
/* usagePeriod */ null, /* isSingleWifi */ false);
|
/* usagePeriod */ null, /* isSingleWifi */ false);
|
||||||
} else {
|
} else {
|
||||||
info = mDataUsageController.getDataUsageInfo(
|
|
||||||
NetworkTemplate.buildTemplateWifiWildcard());
|
|
||||||
summaryPreference.setWifiMode(/* isWifiMode */ true, /* usagePeriod */
|
summaryPreference.setWifiMode(/* isWifiMode */ true, /* usagePeriod */
|
||||||
info.period, /* isSingleWifi */ false);
|
info.period, /* isSingleWifi */ false);
|
||||||
summaryPreference.setLimitInfo(null);
|
summaryPreference.setLimitInfo(null);
|
||||||
|
Reference in New Issue
Block a user