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

Issue 889616 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 768748
Owner:
Closed: Sep 28
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

CrOS notification badge images are not shown correctly

Project Member Reported by jlklein@chromium.org, Sep 26

Issue description

Chrome Version: 71.0.3554.0 (Official Build) dev (64-bit). Also repros on M70.
OS: ChromeOS 11078.0.0

What steps will reproduce the problem?
(1) Go to https://tests.peter.sh/notification-generator/
(2) Show a notification with a badge image

What is the expected result?
Badge icon shows in the top left of the notification next to the domain

What happens instead?
It shows the grey chrome icon there instead.



Please use labels and text to provide additional information.

If this is a regression (i.e., worked before), please consider using the
bisect tool (https://www.chromium.org/developers/bisect-builds-py) to help
us identify the root cause and more rapidly triage the issue.

For graphics-related bugs, please copy/paste the contents of the about:gpu
page at the end of this report.


 
Screenshot 2018-09-26 at 2.56.55 PM.png
1.6 MB View Download
Well that makes sense, thanks Evan! Peter, have notifications from websites always just had the Chrome icon on the top left instead of being configurable?

Sebastian, I assume this isn't the intended behavior on ChromeOS, is it?
I'm a bit blurry because the spec dates back a year now but I believe this was implemented because Websites cannot/do not provide proper badge resources. Ideally websites would follow Android's requirement for badge so we can scale it appropriately. When we have this, we can align.
A year ago, Peter tried to make a similar change. I don't remember the reason why we ended up not doing this.

https://crrev.com/c/684454
 https://crbug.com/768748 

Even if the change is landed, it fills the favicon in grey, so maybe it's still different from what SMS sync team expects.
Mergedinto: 768748
Status: Duplicate (was: Untriaged)
Thanks, tetsui. This does sound like a dupe of crbug/768748. Let's move the conversation there.

Sign in to add a comment