New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug

Blocked on:
issue 542232
issue 571056



Sign in to add a comment

Add finch control for auto-dismiss timeout of notifications

Project Member Reported by owe...@chromium.org, Sep 11 2015

Issue description

We should add a finch control to the auto-dismiss timeout of notifications so we can adjust it without a 6-12 week delay.
 

Comment 1 by peter@chromium.org, Sep 11 2015

Cc: dewittj@chromium.org
+dewittj

The current default is 8 seconds. Justin, do you remember how this was chosen? Experiments done by a partner seem to indicate that 15-20s has the best CTR.
Not sure if there was any actual data for this choice, beyond comparisons to other existing notification systems at the time.  Note that priority=1 notifications have a timeout of something like 21 seconds, so that could also be controlled by finch.
Also I am happy to reconfigure the defaults based on data from partners or otherwise.

Comment 4 by peter@chromium.org, Sep 11 2015

Status: Available
One thing I particularly care about is predictability - if all notifications stay on-screen for either 8 or 25 seconds, *except* for this one kind of notification, it's just going to be annoying for users.

For this reason I think doing an experiment makes sense. I think the most useful metric for A/B testing are the number of manually dismissed notifications; clicks will naturally rise the longer a notification is on screen.

Comment 5 by peter@chromium.org, Sep 29 2015

Labels: M-48
Status: Assigned
We'll increase the timeout for M47, but really want to run our own experiments in M48.
Labels: Merge-Request-47
Requesting merge of https://codereview.chromium.org/1395483002 to m47. It's a safe patch, and will avoid developers having to implement hackish workarounds that will break in future version of Chrome (risking compatibility issues).

Comment 8 by tin...@google.com, Oct 8 2015

Labels: -Merge-Request-47 Merge-Approved-47 Hotlist-Merge-Approved
Approved for M47 (branch: 2526)
Project Member

Comment 9 by bugdroid1@chromium.org, Oct 9 2015

Labels: -Merge-Approved-47 merge-merged-2526
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b2fa718b0cff37c436964a52c75a78d4fe538b29

commit b2fa718b0cff37c436964a52c75a78d4fe538b29
Author: John Mellor <johnme@chromium.org>
Date: Fri Oct 09 10:36:16 2015

Increase Web Notification timeout to 20 seconds

Data from partners suggests this improves re-engagement. We intend to
revisit this in m48 with a Finch experiment.

BUG= 530697 

Review URL: https://codereview.chromium.org/1395483002

Cr-Commit-Position: refs/heads/master@{#352904}
(cherry picked from commit 07e31a310e07d3aa8a07c45670e70e89094317b7)

Review URL: https://codereview.chromium.org/1395103002 .

Cr-Commit-Position: refs/branch-heads/2526@{#57}
Cr-Branched-From: cb947c0153db0ec02a8abbcb3ca086d88bf6006f-refs/heads/master@{#352221}

[modify] http://crrev.com/b2fa718b0cff37c436964a52c75a78d4fe538b29/chrome/browser/notifications/platform_notification_service_impl.cc
[modify] http://crrev.com/b2fa718b0cff37c436964a52c75a78d4fe538b29/ui/message_center/message_center_style.cc
[modify] http://crrev.com/b2fa718b0cff37c436964a52c75a78d4fe538b29/ui/message_center/message_center_style.h
[modify] http://crrev.com/b2fa718b0cff37c436964a52c75a78d4fe538b29/ui/message_center/notification.cc
[modify] http://crrev.com/b2fa718b0cff37c436964a52c75a78d4fe538b29/ui/message_center/notification.h
[modify] http://crrev.com/b2fa718b0cff37c436964a52c75a78d4fe538b29/ui/message_center/popup_timer.cc

Project Member

Comment 10 by bugdroid1@chromium.org, Oct 9 2015

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/bling/chromium.git/+/b2fa718b0cff37c436964a52c75a78d4fe538b29

commit b2fa718b0cff37c436964a52c75a78d4fe538b29
Author: John Mellor <johnme@chromium.org>
Date: Fri Oct 09 10:36:16 2015

Blockedon: chromium:542232
Project Member

Comment 12 by bugdroid1@chromium.org, Oct 20 2015

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f272cc7c00097e01454be7f7f370516b52989b7f

commit f272cc7c00097e01454be7f7f370516b52989b7f
Author: johnme <johnme@chromium.org>
Date: Tue Oct 20 13:51:40 2015

Add use counter for user closing a persistent notification.

Adds a Notifications.Persistent.ClosedByUser use counter.

This is a follow-up to f52d070a3cae53f27cf9490428770b64e73c16d0
(https://codereview.chromium.org/1315193011) which added a
Notifications.Persistent.Closed use counter that included both closing
by user and by system.

BUG= 530697 

Review URL: https://codereview.chromium.org/1401303003

Cr-Commit-Position: refs/heads/master@{#355050}

[modify] http://crrev.com/f272cc7c00097e01454be7f7f370516b52989b7f/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationUIManager.java
[modify] http://crrev.com/f272cc7c00097e01454be7f7f370516b52989b7f/chrome/browser/notifications/notification_ui_manager_android.cc
[modify] http://crrev.com/f272cc7c00097e01454be7f7f370516b52989b7f/chrome/browser/notifications/notification_ui_manager_android.h
[modify] http://crrev.com/f272cc7c00097e01454be7f7f370516b52989b7f/chrome/browser/notifications/persistent_notification_delegate.cc
[modify] http://crrev.com/f272cc7c00097e01454be7f7f370516b52989b7f/chrome/browser/notifications/platform_notification_service_impl.cc
[modify] http://crrev.com/f272cc7c00097e01454be7f7f370516b52989b7f/chrome/browser/notifications/platform_notification_service_impl.h
[modify] http://crrev.com/f272cc7c00097e01454be7f7f370516b52989b7f/chrome/browser/push_messaging/push_messaging_notification_manager.cc
[modify] http://crrev.com/f272cc7c00097e01454be7f7f370516b52989b7f/tools/metrics/actions/actions.xml

Blockedon: chromium:571056
Labels: -Pri-2 -M-48 Pri-3
Owner: ----
Status: Available
Deprioritizing Finch experiment as 20 seconds seems to be working well so far, and efforts like  issue 571056  might make it redundant.

Comment 14 by peter@chromium.org, Jan 12 2016

Is that statement based on (the lack of) user reports? Do we have data?

If we feel that this is an appropriate amount of time, let's make sure that we change extensions (kAutocloseHighPriorityDelaySeconds) to match this and close this issue.
So lets say someone walks away from their computer then comes back to no notifications? Do you see the problem here? Our chat application's user experience is suffering due to 20 second timeout on notifications. Thoughts?
Project Member

Comment 16 by sheriffbot@chromium.org, Apr 21 2016



For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: -mvanouwe...@chromium.org
Owner: peter@chromium.org
Status: Assigned (was: Available)
High-priority and Web notifications should just use the same value. CL uploaded:
https://chromium-review.googlesource.com/c/559102/
Project Member

Comment 19 by bugdroid1@chromium.org, Jul 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fd9298e768838d6f0adfffcbd5733fa627edebb0

commit fd9298e768838d6f0adfffcbd5733fa627edebb0
Author: Peter Beverloo <peter@chromium.org>
Date: Fri Jul 07 17:48:56 2017

Align on-screen time of high priority and Web notifications

These used to remain on-screen for respectively 25 and 20 seconds, the
difference in which is a bit silly. Let's both keep them on-screen for
25 seconds.

BUG= 530697 

Change-Id: I082ecfc69ec69e600818b113b02e161e058a830f
Reviewed-on: https://chromium-review.googlesource.com/559102
Commit-Queue: Peter Beverloo <peter@chromium.org>
Reviewed-by: John Mellor <johnme@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#484976}
[modify] https://crrev.com/fd9298e768838d6f0adfffcbd5733fa627edebb0/ui/message_center/message_center_impl_unittest.cc
[modify] https://crrev.com/fd9298e768838d6f0adfffcbd5733fa627edebb0/ui/message_center/message_center_style.h
[modify] https://crrev.com/fd9298e768838d6f0adfffcbd5733fa627edebb0/ui/message_center/popup_timers_controller.cc

Comment 20 by peter@chromium.org, Jul 11 2017

Status: Fixed (was: Assigned)

Sign in to add a comment