From c6ecbce42b8a529447ff327a7902d9da0aeb4310 Mon Sep 17 00:00:00 2001
From: Amith Yamasani <yamasani@google.com>
Date: Tue, 23 Jun 2015 12:58:43 -0700
Subject: [PATCH] Clarify the intention of setNotificationsShown

Updated javadoc with a bit more detail.

Bug: 21572672
Change-Id: I6b24e47fecd5142bb4bac979cb91f7a6359990d5
---
 .../service/notification/NotificationListenerService.java     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/core/java/android/service/notification/NotificationListenerService.java b/core/java/android/service/notification/NotificationListenerService.java
index 0309d24c8e565..d424546a9233a 100644
--- a/core/java/android/service/notification/NotificationListenerService.java
+++ b/core/java/android/service/notification/NotificationListenerService.java
@@ -369,7 +369,9 @@ public abstract class NotificationListenerService extends Service {
 
     /**
      * Inform the notification manager that these notifications have been viewed by the
-     * user.
+     * user. This should only be called when there is sufficient confidence that the user is
+     * looking at the notifications, such as when the notifications appear on the screen due to
+     * an explicit user interaction.
      * @param keys Notifications to mark as seen.
      */
     public final void setNotificationsShown(String[] keys) {