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

Merge "Decouple logUIevent from Bubble class" into rvc-dev am: bcbcca7d4f

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11849441

Change-Id: I1877094e064b1841afa3239f564b445610e456b1
This commit is contained in:
TreeHugger Robot
2020-06-18 14:53:34 +00:00
committed by Automerger Merge Worker
parent a4003283b8 bcbcca7d4f
commit 5993341a6a
1 changed files with 20 additions and 25 deletions
Download Patch File Download Diff File Expand all files Collapse all files

45
packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java
View File

@@ -51,6 +51,7 @@ import android.graphics.drawable.TransitionDrawable;
import android.os.Bundle;
import android.os.Handler;
import android.provider.Settings;
import android.service.notification.StatusBarNotification;
import android.util.Log;
import android.view.Choreographer;
import android.view.DisplayCutout;
@@ -1065,10 +1066,8 @@ public class BubbleStackView extends FrameLayout
if (bubble != null && mBubbleData.hasBubbleInStackWithKey(bubble.getKey())) {
final Intent intent = bubble.getSettingsIntent(mContext);
collapseStack(() -> {
mContext.startActivityAsUser(intent, bubble.getUser());
logBubbleClickEvent(
bubble,
logBubbleEvent(bubble,
SysUiStatsLog.BUBBLE_UICHANGED__ACTION__HEADER_GO_TO_SETTINGS);
});
}
@@ -2757,16 +2756,28 @@ public class BubbleStackView extends FrameLayout
/**
* Logs the bubble UI event.
*
* @param bubble the bubble that is being interacted on. Null value indicates that
* the user interaction is not specific to one bubble.
* @param provider the bubble view provider that is being interacted on. Null value indicates
* that the user interaction is not specific to one bubble.
* @param action the user interaction enum.
*/
private void logBubbleEvent(@Nullable BubbleViewProvider bubble, int action) {
if (bubble == null) {
private void logBubbleEvent(@Nullable BubbleViewProvider provider, int action) {
if (provider == null || provider.getKey().equals(BubbleOverflow.KEY)) {
SysUiStatsLog.write(SysUiStatsLog.BUBBLE_UI_CHANGED,
mContext.getApplicationInfo().packageName,
provider == null ? null : BubbleOverflow.KEY /* notification channel */,
0 /* notification ID */,
0 /* bubble position */,
getBubbleCount(),
action,
getNormalizedXPosition(),
getNormalizedYPosition(),
false /* unread bubble */,
false /* on-going bubble */,
false /* isAppForeground (unused) */);
return;
}
bubble.logUIEvent(getBubbleCount(), action, getNormalizedXPosition(),
getNormalizedYPosition(), getBubbleIndex(bubble));
provider.logUIEvent(getBubbleCount(), action, getNormalizedXPosition(),
getNormalizedYPosition(), getBubbleIndex(provider));
}
/**
@@ -2805,20 +2816,4 @@ public class BubbleStackView extends FrameLayout
}
return bubbles;
}
/**
* Logs bubble UI click event.
*
* @param bubble the bubble notification entry that user is interacting with.
* @param action the user interaction enum.
*/
private void logBubbleClickEvent(Bubble bubble, int action) {
bubble.logUIEvent(
getBubbleCount(),
action,
getNormalizedXPosition(),
getNormalizedYPosition(),
getBubbleIndex(getExpandedBubble())
);
}
}