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

Issue 670060 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Screen resolution change notifications are not deduped

Project Member Reported by zelidrag@chromium.org, Nov 30 2016

Issue description

Chrome Version       : 57.0.2933.0 (canary)

Screen resolution change notifications are not deduped if there are other  notification from Android apps present. 

What steps will reproduce the problem?
1. Have an Android notification that isn't dismissed yet. 
2. Press Ctrl+Shift+'+' several times all the way to the end
3. Press Ctrl+Shift+'-' all the way to the end
4. Keep repeating (3) and (4) few times

What is the expected result?

There should be only one pending screen resolution change notification display in the notification center.

What happens instead of that?

Screen resolution change notifications keep piling - it seems always when Android notifications are present. See attached screenshots.
 
Screenshot 2016-11-30 at 1.26.48 PM.png
2.9 MB View Download
Screenshot 2016-11-30 at 1.27.04 PM.png
2.9 MB View Download
Cc: dewittj@chromium.org
+dewittj for notification issue, do you have any idea about recent change?
Status: Started (was: Assigned)
Ok I can repro. It looks same ID notifications are not handled correctly in the message center.
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 6 2016

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

commit 6aea3f6ecd4f2ced5ffdf88196ccb59e469723aa
Author: yoshiki <yoshiki@chromium.org>
Date: Tue Dec 06 18:03:29 2016

Remove newly-added notification correctly in the message center

BUG= 670060 
TEST=manual tested

Review-Url: https://codereview.chromium.org/2544533005
Cr-Commit-Position: refs/heads/master@{#436641}

[modify] https://crrev.com/6aea3f6ecd4f2ced5ffdf88196ccb59e469723aa/ui/message_center/views/message_list_view.cc

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Verified on 57.0.2949.0/9081.0.0

Sign in to add a comment