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

Fix NPE issue of usage detailed page learn more button click event

Browse Source 
root cause: some OEMs can not provide valid intent for help & feedback
solution: check the intent is vaild or not before show up the help content

Bug: 204844010
Test: make RunSettingsRoboTests
Change-Id: I4e7a7c926205be1179d55d33ada345024c8a44ab
This commit is contained in:
ykhung
2021-11-02 23:40:00 +08:00
committed by YUKAI HUNG
parent ae5bddc5e5
commit 8f0291cd1a
1 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/com/android/settings/fuelgauge/AdvancedPowerUsageDetail.java
View File

@@ -383,12 +383,14 @@ public class AdvancedPowerUsageDetail extends DashboardFragment implements
footerString = context.getString(R.string.manager_battery_usage_footer); footerString = context.getString(R.string.manager_battery_usage_footer);
} }
mFooterPreference.setTitle(footerString); mFooterPreference.setTitle(footerString);
mFooterPreference.setLearnMoreAction(v -> final Intent helpIntent = HelpUtils.getHelpIntent(context, context.getString(
startActivityForResult(HelpUtils.getHelpIntent(context, R.string.help_url_app_usage_settings), /*backupContext=*/ "");
context.getString(R.string.help_url_app_usage_settings), if (helpIntent != null) {
/*backupContext=*/ ""), /*requestCode=*/ 0)); mFooterPreference.setLearnMoreAction(v ->
mFooterPreference.setLearnMoreContentDescription( startActivityForResult(helpIntent, /*requestCode=*/ 0));
context.getString(R.string.manager_battery_usage_link_a11y)); mFooterPreference.setLearnMoreContentDescription(
context.getString(R.string.manager_battery_usage_link_a11y));
}
} }
@Override @Override