1225845 - The notifications.notify API and Firefox version 44.0a2 icon size is larger than previous versions

Status: NEW

(Toolkit :: Alerts Service, defect)

Description

[daz]

Attachment: 44.0a2 bottom and 42 top

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:44.0) Gecko/20100101 Firefox/44.0
Build ID: 20151117004023

Steps to reproduce:

Used an add-on which uses the "notifications.notify" API


Actual results:

The notification icon in Firefox version 44.0a2 is larger than in previous Firefox versions such as 43.0b3


Expected results:

The icon should not have changed in size (no coding has changed within the add-on)

Comment 1

[Boris Zbarsky [:bzbarsky]]

The icon is clearly not part of the core DOM...

Comment 2

[daz]

I am new at this so please keep that in mind. So this is not a bug as such which should have been added here, is that correct?

Comment 3

[Paul Silaghi, QA [:pauly]]

Can you please provide some steps to reproduce and also a testcase to show the problem ?

Comment 4

[daz]

Have 2 versions of Firefox, one v 44.0a2 and the other 42. Install an add-on which will utilize the notifications.notify API such as HTTPSPATROL. Go to Google.com for example an each Firefox version and you can see the difference. Version 42 will be the correct display of the notification. Also see the images provided for guidance.

Please also see the following Forum Discussion on others who are experiencing the same issue:

https://discourse.mozilla-community.org/t/add-on-notifications-notify-api-and-firefox-44-0a2/5157

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

max-width and max-height rules for the icon got removed at https://hg.mozilla.org/mozilla-central/rev/6ecebbea7718#l11.66

Comment 6

[daz]

(In reply to Sebastian H. [:aryx][:archaeopteryx] from comment #5)
> max-width and max-height rules for the icon got removed at
> https://hg.mozilla.org/mozilla-central/rev/6ecebbea7718#l11.66

Thanks for this.. The removal has caused some add-on to change unexpectedly.

Can the max-height and max-width be added manually?