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

Merge "ConditionManager: don't crash when trying to add unexpected condition"

Browse Source
This commit is contained in:
Treehugger Robot
2017-06-03 23:49:37 +00:00
committed by Gerrit Code Review
parent c7697e9b8a f7791d75bf
commit 46f7199cb1
1 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/com/android/settings/dashboard/conditional/ConditionManager.java
View File

@@ -91,8 +91,12 @@ public class ConditionManager {
Condition condition = createCondition(Class.forName(clz));
PersistableBundle bundle = PersistableBundle.restoreFromXml(parser);
if (DEBUG) Log.d(TAG, "Reading " + clz + " -- " + bundle);
if (condition != null) {
condition.restoreState(bundle);
conditions.add(condition);
} else {
Log.e(TAG, "failed to add condition: " + clz);
}
while (parser.getDepth() > depth) {
parser.next();
}
@@ -150,7 +154,10 @@ public class ConditionManager {
private void addIfMissing(Class<? extends Condition> clz, ArrayList<Condition> conditions) {
if (getCondition(clz, conditions) == null) {
if (DEBUG) Log.d(TAG, "Adding missing " + clz.getName());
conditions.add(createCondition(clz));
Condition condition = createCondition(clz);
if (condition != null) {
conditions.add(condition);
}
}
}
@@ -172,7 +179,8 @@ public class ConditionManager {
} else if (NightDisplayCondition.class == clz) {
return new NightDisplayCondition(this);
}
throw new RuntimeException("Unexpected Condition " + clz);
Log.e(TAG, "unknown condition class: " + clz.getSimpleName());
return null;
}
Context getContext() {