Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactors logging into its own function

Browse Source 
This will allow logging behavior to be overridden.

Test: Tested locally
BUG:128982146
Change-Id: Iaabc4ad0b44147732e05b4b9317e49bcee91683d
This commit is contained in:
Govinda Wasserman
2019-06-03 12:37:43 -04:00
parent 47350b01b4
commit 6418e974a1
1 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
packages/SystemUI/src/com/android/systemui/assist/AssistManager.java
View File

@@ -259,11 +259,7 @@ public class AssistManager implements ConfigurationChangedReceiver {
int phoneState = mPhoneStateMonitor.getPhoneState();
args.putInt(INVOCATION_PHONE_STATE_KEY, phoneState);
args.putLong(INVOCATION_TIME_MS_KEY, SystemClock.uptimeMillis());
// Logs assistant start with invocation type.
MetricsLogger.action(
new LogMaker(MetricsEvent.ASSISTANT)
.setType(MetricsEvent.TYPE_OPEN)
.setSubtype(toLoggingSubType(invocationType, phoneState)));
logStartAssist(invocationType, phoneState);
startAssistInternal(args, assistComponent, isService);
}
@@ -449,7 +445,14 @@ public class AssistManager implements ConfigurationChangedReceiver {
return toLoggingSubType(invocationType, mPhoneStateMonitor.getPhoneState());
}
private int toLoggingSubType(int invocationType, int phoneState) {
protected void logStartAssist(int invocationType, int phoneState) {
MetricsLogger.action(
new LogMaker(MetricsEvent.ASSISTANT)
.setType(MetricsEvent.TYPE_OPEN)
.setSubtype(toLoggingSubType(invocationType, phoneState)));
}
protected final int toLoggingSubType(int invocationType, int phoneState) {
// Note that this logic will break if the number of Assistant invocation types exceeds 7.
// There are currently 5 invocation types, but we will be migrating to the new logging
// framework in the next update.