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

Merge "[Notif] Limit shown events to manager.show" into pi-dev

Browse Source
This commit is contained in:
TreeHugger Robot
2018-05-10 17:40:03 +00:00
committed by Android (Google) Code Review
parent 5a59a71d40 eab86045e8
commit 0ec2e0885c
2 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
packages/SystemUI/src/com/android/systemui/statusbar/NotificationBlockingHelperManager.java
View File

@@ -23,8 +23,10 @@ import android.service.notification.StatusBarNotification;
import android.support.annotation.VisibleForTesting;
import android.util.Log;
import com.android.internal.logging.MetricsLogger;
import com.android.systemui.Dependency;
import com.android.systemui.plugins.statusbar.NotificationMenuRowPlugin;
import com.android.systemui.statusbar.notification.NotificationCounters;
import com.android.systemui.statusbar.phone.StatusBar;
import java.util.Collections;
@@ -97,6 +99,9 @@ public class NotificationBlockingHelperManager {
// We don't care about the touch origin (x, y) since we're opening guts without any
// explicit user interaction.
manager.openGuts(mBlockingHelperRow, 0, 0, menuRow.getLongpressMenuItem(mContext));
Dependency.get(MetricsLogger.class)
.count(NotificationCounters.BLOCKING_HELPER_SHOWN, 1);
return true;
}
return false;

2
packages/SystemUI/src/com/android/systemui/statusbar/NotificationInfo.java
View File

@@ -199,8 +199,6 @@ public class NotificationInfo extends LinearLayout implements NotificationGuts.G
bindHeader();
bindPrompt();
bindButtons();
logBlockingHelperCounter(NotificationCounters.BLOCKING_HELPER_SHOWN);
}
private void bindHeader() throws RemoteException {