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

Issue 598636 link

Starred by 1 user
Status: WontFix
Owner:
dewittj@chromium.org
Closed: Mar 2016
Cc:
nyerramilli@chromium.org
█ ranjitkan@chromium.org
peter@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Unnecessary notification bubble of ‘MightyText’ stay for long time evenafter click on it.

Reported by dmascare...@etouch.net, Mar 29 2016 
Chrome Version:51.0.2693.2 Revision 255cb9d4b581ae3eb9a5b582175bc18c7997a741-refs/branch-heads/2693@{#2}(32/64 bit)
OS: Mac, Windows

Test url: https://chrome.google.com/webstore/detail/mightytext-sms-from-pc-te/dkfhfaphfkopdgpbfkebjfcblcafcmpi/related?hl=en

What steps will reproduce the problem?
1. Launch chrome and navigate to above url 
2. Click on ‘Add to chrome’ and let the notification bubble appears 
3. Click on the notification bubble (multiple times) and observe.

Actual: Unnecessary notification bubble of ‘MightyText’ stay for long time evenafter click on it and multiple ‘Sign in’ page gets open after step 3.
Expected: Notification bubble should disappears after first click on it and single ‘sign in’ page should get opened after step 3.

This is regression issue,broken in ‘M 47’ and below is narrow bisect:
https://chromium.googlesource.com/chromium/src/+log/05608da225e05b4a20da3f4ca0cb9a5276fcf3d6..ca1bc796c396b669c8c20d6487e16c63505c9b73

Suspecting: r350945 ?

Good build:47.0.2519.0
Bad build:47.0.2520.0

Could you please help to reassign if your change is not the cause for this change?

Comment 1 by dmascare...@etouch.net, Mar 29 2016 

Please find the attached video.
Actual_MT.mov
4.4 MB Download
Expected_MT.mov
1.4 MB Download

Comment 2 by peter@chromium.org, Mar 29 2016

Status: WontFix (was: Assigned)
Thank you for the report!

Their chrome.notifications.onClicked listener creates a new tab, but doesn't actually close the notification (in this case the "user_not_logged_in" notification in //scripts/notifications.js).

Previously this would cause notifications to dismiss to the notification center (where it would remain available), but since we removed the notification center from the non-Chrome OS platforms in M47 they now stay on screen.

The developer should close the notification when it no longer is needed. This is WAI from our side.
►

Sign in to add a comment