commit 2e8dc1c49fb02addc8574c46c8864fc5307f3bc6
author TreeHugger Robot <treehugger-gerrit@google.com> Wed Dec 07 11:24:15 2022 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Wed Dec 07 11:24:15 2022 +0000
tree b1a707393e8b3308e7f27d7a008cb13c569fd3bf
parent d73c751f3df29439f9d8df8b6133da816c0548d9
parent 76f19809c13c650e328c2f30033ff7a3b7312aeb

Merge "Fix b/261390301: NullPointerException at BatteryChartPreferenceController.addAllPreferences()"

src/com/android/settings/fuelgauge/batteryusage/BatteryChartPreferenceController.java

diff --git a/src/com/android/settings/fuelgauge/batteryusage/BatteryChartPreferenceController.java b/src/com/android/settings/fuelgauge/batteryusage/BatteryChartPreferenceController.java
index 707c5b2..c2eab57 100644
--- a/src/com/android/settings/fuelgauge/batteryusage/BatteryChartPreferenceController.java
+++ b/src/com/android/settings/fuelgauge/batteryusage/BatteryChartPreferenceController.java
@@ -319,7 +319,8 @@
         }
 
 
-        if (mOnBatteryUsageUpdatedListener != null) {
+        if (mOnBatteryUsageUpdatedListener != null && mBatteryUsageMap != null
+                && mBatteryUsageMap.get(mDailyChartIndex) != null) {
             final BatteryDiffData slotUsageData =
                     mBatteryUsageMap.get(mDailyChartIndex).get(mHourlyChartIndex);
             mOnBatteryUsageUpdatedListener.onBatteryUsageUpdated(
@@ -359,6 +360,9 @@
             mDailyChartIndex = 0;
         } else {
             mDailyChartView.setVisibility(View.VISIBLE);
+            if (mDailyChartIndex >= mDailyViewModel.size()) {
+                mDailyChartIndex = BatteryChartViewModel.SELECTED_INDEX_ALL;
+            }
             mDailyViewModel.setSelectedIndex(mDailyChartIndex);
             mDailyChartView.setViewModel(mDailyViewModel);
         }
@@ -370,6 +374,9 @@
             animateBatteryHourlyChartView(/*visible=*/ true);
             final BatteryChartViewModel hourlyViewModel =
                     mHourlyViewModels.get(mDailyChartIndex);
+            if (mHourlyChartIndex >= hourlyViewModel.size()) {
+                mHourlyChartIndex = BatteryChartViewModel.SELECTED_INDEX_ALL;
+            }
             hourlyViewModel.setSelectedIndex(mHourlyChartIndex);
             mHourlyChartView.setViewModel(hourlyViewModel);
         }