Exclude charging period when calculate the reattribute duration (5/5)

Bug: 346706894
Test: atest SettingsRoboTests:com.android.settings.fuelgauge.batteryusage
Flag: EXEMPT bug fix
Change-Id: Icc9a475a71f189e72bf06f9a0b4c23380a90a603
This commit is contained in:
YK Hung
2024-06-29 07:21:58 +00:00
parent af053aa3cc
commit 29f73d68fe
3 changed files with 6 additions and 2 deletions

View File

@@ -26,6 +26,7 @@ import android.util.SparseIntArray;
import androidx.annotation.NonNull; import androidx.annotation.NonNull;
import com.android.settings.fuelgauge.batteryusage.BatteryDiffData; import com.android.settings.fuelgauge.batteryusage.BatteryDiffData;
import com.android.settings.fuelgauge.batteryusage.BatteryEvent;
import com.android.settings.fuelgauge.batteryusage.DetectRequestSourceType; import com.android.settings.fuelgauge.batteryusage.DetectRequestSourceType;
import com.android.settings.fuelgauge.batteryusage.PowerAnomalyEventList; import com.android.settings.fuelgauge.batteryusage.PowerAnomalyEventList;
import com.android.settingslib.fuelgauge.Estimate; import com.android.settingslib.fuelgauge.Estimate;
@@ -166,5 +167,6 @@ public interface PowerUsageFeatureProvider {
boolean processBatteryReattributeData( boolean processBatteryReattributeData(
@NonNull Context context, @NonNull Context context,
@NonNull Map<Long, BatteryDiffData> batteryDiffDataMap, @NonNull Map<Long, BatteryDiffData> batteryDiffDataMap,
@NonNull List<BatteryEvent> batteryEventList,
final boolean isFromPeriodJob); final boolean isFromPeriodJob);
} }

View File

@@ -18,7 +18,6 @@ package com.android.settings.fuelgauge;
import static com.android.settings.Utils.SYSTEMUI_PACKAGE_NAME; import static com.android.settings.Utils.SYSTEMUI_PACKAGE_NAME;
import android.annotation.Nullable;
import android.content.Context; import android.content.Context;
import android.content.Intent; import android.content.Intent;
import android.content.pm.PackageManager; import android.content.pm.PackageManager;
@@ -28,9 +27,11 @@ import android.util.ArraySet;
import android.util.SparseIntArray; import android.util.SparseIntArray;
import androidx.annotation.NonNull; import androidx.annotation.NonNull;
import androidx.annotation.Nullable;
import com.android.internal.util.ArrayUtils; import com.android.internal.util.ArrayUtils;
import com.android.settings.fuelgauge.batteryusage.BatteryDiffData; import com.android.settings.fuelgauge.batteryusage.BatteryDiffData;
import com.android.settings.fuelgauge.batteryusage.BatteryEvent;
import com.android.settings.fuelgauge.batteryusage.DetectRequestSourceType; import com.android.settings.fuelgauge.batteryusage.DetectRequestSourceType;
import com.android.settings.fuelgauge.batteryusage.PowerAnomalyEventList; import com.android.settings.fuelgauge.batteryusage.PowerAnomalyEventList;
import com.android.settingslib.fuelgauge.Estimate; import com.android.settingslib.fuelgauge.Estimate;
@@ -254,6 +255,7 @@ public class PowerUsageFeatureProviderImpl implements PowerUsageFeatureProvider
public boolean processBatteryReattributeData( public boolean processBatteryReattributeData(
@NonNull Context context, @NonNull Context context,
@NonNull Map<Long, BatteryDiffData> batteryDiffDataMap, @NonNull Map<Long, BatteryDiffData> batteryDiffDataMap,
@NonNull List<BatteryEvent> batteryEventList,
final boolean isFromPeriodJob) { final boolean isFromPeriodJob) {
return false; return false;
} }

View File

@@ -522,7 +522,7 @@ public class DataProcessManager {
FeatureFactory.getFeatureFactory() FeatureFactory.getFeatureFactory()
.getPowerUsageFeatureProvider(); .getPowerUsageFeatureProvider();
featureProvider.processBatteryReattributeData( featureProvider.processBatteryReattributeData(
mContext, batteryDiffDataMap, mIsFromPeriodJob); mContext, batteryDiffDataMap, mBatteryEventList, mIsFromPeriodJob);
Log.d( Log.d(
TAG, TAG,