d3d49337e55c70d61358e028d879dea4d95f8014 - SHIFTPHONES/android_frameworks_base

commit	d3d49337e55c70d61358e028d879dea4d95f8014	[log] [tgz]
author	Will Brockman <brockman@google.com>	Mon Feb 10 19:43:03 2020 -0500
committer	Will Brockman <brockman@google.com>	Wed Feb 12 16:13:46 2020 -0500
tree	9eac873c1a6fddf774a83c2f1865a6d4dd797db6
parent	e5bbdcfcf5dac8e3c4dec8d97639710c55dd76a7 [diff]

Notification statsd logs: Visibility changes.

These are currently logged in NotificationRecord.setVisibility().
Rather than add a dependency on the NotificationRecordLogger to
NotificationRecord, I've passed one in to that method.

I considered but rejected doing the same for
NOTIFICATION_INTERRUPTION, which is logged similarly in
NotificationRecord.setInterruptive().  The circumstances under which
that log gets written are challenging to understand, and I'm not
currently convinced that it is worth porting.

Test: atest NotificationManagerServiceTest NotificiationRecordLoggerTest
Bug: 146488473
Change-Id: I956cd4ededa470f91244305d4c26fb981d4c94a4

services/core/java/com/android/server/notification/NotificationManagerService.java[diff]
services/core/java/com/android/server/notification/NotificationRecord.java[diff]
services/core/java/com/android/server/notification/NotificationRecordLogger.java[diff]
services/core/java/com/android/server/notification/NotificationRecordLoggerImpl.java[diff]
services/tests/uiservicestests/src/com/android/server/notification/NotificationManagerServiceTest.java[diff]
services/tests/uiservicestests/src/com/android/server/notification/NotificationRecordLoggerFake.java[diff]

6 files changed