PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Reorder default app summary to match subpage ordering

Browse Source 
Change-Id: I618f44ba8cf6f590b86d483cfecb090b71ef1268
Fixes: 65049890
Test: robotests
This commit is contained in:
Fan Zhang
2017-10-16 16:42:27 -07:00
parent 7ad17d72ad
commit 11e9123036
2 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/com/android/settings/applications/DefaultAppSettings.java
View File

@@ -133,10 +133,10 @@ public class DefaultAppSettings extends DashboardFragment {
return;
}
CharSequence summary = concatSummaryText(
mDefaultSmsPreferenceController.getDefaultAppLabel(),
mDefaultBrowserPreferenceController.getDefaultAppLabel());
summary = concatSummaryText(summary,
mDefaultBrowserPreferenceController.getDefaultAppLabel(),
mDefaultPhonePreferenceController.getDefaultAppLabel());
summary = concatSummaryText(summary,
mDefaultSmsPreferenceController.getDefaultAppLabel());
if (!TextUtils.isEmpty(summary)) {
mSummaryLoader.setSummary(this, summary);
}

6
tests/robotests/src/com/android/settings/applications/DefaultAppSettingsTest.java
View File

@@ -95,7 +95,7 @@ public class DefaultAppSettingsTest {
when(defaultBrowser.getDefaultAppLabel()).thenReturn("Browser1");
when(defaultPhone.getDefaultAppLabel()).thenReturn("Phone1");
summaryProvider.setListening(true);
verify(summaryLoader).setSummary(summaryProvider, "Sms1, Browser1, Phone1");
verify(summaryLoader).setSummary(summaryProvider, "Browser1, Phone1, Sms1");
// 2 available
when(defaultSms.getDefaultAppLabel()).thenReturn(null);
@@ -108,13 +108,13 @@ public class DefaultAppSettingsTest {
when(defaultBrowser.getDefaultAppLabel()).thenReturn(null);
when(defaultPhone.getDefaultAppLabel()).thenReturn("Phone1");
summaryProvider.setListening(true);
verify(summaryLoader).setSummary(summaryProvider, "Sms1, Phone1");
verify(summaryLoader).setSummary(summaryProvider, "Phone1, Sms1");
when(defaultSms.getDefaultAppLabel()).thenReturn("Sms1");
when(defaultBrowser.getDefaultAppLabel()).thenReturn("Browser1");
when(defaultPhone.getDefaultAppLabel()).thenReturn(null);
summaryProvider.setListening(true);
verify(summaryLoader).setSummary(summaryProvider, "Sms1, Browser1");
verify(summaryLoader).setSummary(summaryProvider, "Phone1, Sms1");
// 1 available
when(defaultSms.getDefaultAppLabel()).thenReturn(null);