Closed
Bug 1215861
Opened 9 years ago
Closed 8 years ago
Notifications popup does not work the first time with iconURL:remote image
Categories
(Core :: Graphics: ImageLib, defect)
Tracking
()
People
(Reporter: alice0775, Unassigned)
References
Details
(Keywords: regression, Whiteboard: [dupe me][gfx-noted])
Attachments
(1 file)
1.69 KB,
application/x-xpinstall
|
Details |
I found a bug when I test Bug 1201397.
This problem is since Firefox37 on Linux. (it works on Windows7)
Notifications popup does not work the first time with remote image.
If remove the iconURL:...., It works the first time as expected.
var notifications = require("sdk/notifications");
notifications.notify({
title: "notifications test",
text: "test from sdk",
iconURL: "https://bugzilla.mozilla.org/extensions/BMO/web/producticons/firefox.png"
});
Steps To Reproduce:
1. Install attached addon
2. Restart
--- observe notification
3. Open about:addons
4. Disable and then Enable the addon
--- observe notification
Actual Results:
Notifications popup does not work the 1st time (step.2), but works 2nd, 3rd... time (step.4)
Regression window:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=cac6192956ab&tochange=369a8f14ccf8
Suspect: Bug 1079627
Comment 3•9 years ago
|
||
[Tracking Requested - why for this release]: We are promoting notifications in Fx 42 due to the newly added Push API. It will be confusing to developers if the first notification doesn't work and will reduce trust in this new feature.
tracking-firefox42:
--- → ?
Tracked for 44 since this also impacts push notifications feature to be shipped in FF44.
MattN, will the fix for this come from you/Kit (push notifications team)? If not, who can I ping to get somebody to investigate this? I would consider this as release blocking for push notifications.
Flags: needinfo?(MattN+bmo)
Josh, Andrew: This bug is about losing the first notification with remote images. Could you help find an owner for this bug? Thanks!
Flags: needinfo?(overholt)
Flags: needinfo?(josh)
Comment 9•9 years ago
|
||
Ideally this would come from Seth's team since it's his regression but I think Kit was looking into or fixing this in another bug.
Flags: needinfo?(MattN+bmo)
Comment 10•9 years ago
|
||
I emailed Seth to see what he thinks. Unless Kit can come up with a fix, if this is indeed an ImageLib regression, Seth is likely the best assignee.
Flags: needinfo?(overholt)
We are in RC mode, it's too late and this is now a wontfix for Fx44.
Comment 13•9 years ago
|
||
Milan, who can look at this?
status-firefox45:
--- → wontfix
status-firefox46:
--- → affected
status-firefox47:
--- → affected
status-firefox48:
--- → affected
tracking-firefox44:
+ → ---
Flags: needinfo?(seth) → needinfo?(milan)
Updated•9 years ago
|
Flags: needinfo?(milan)
Updated•9 years ago
|
Flags: needinfo?(howareyou322)
Comment 14•9 years ago
|
||
Does this reproduce anymore? comment 0 worked as expected for me in nightly and release.
Flags: needinfo?(alice0775)
Reporter | ||
Comment 15•9 years ago
|
||
I can still reproduce the problem on Ubuntu14.04.
https://hg.mozilla.org/mozilla-central/rev/9ce31e9f90cb0e534611b0f617c5bbc232ffe748
Mozilla/5.0 (X11; Linux i686; rv:49.0) Gecko/20100101 Firefox/49.0 ID:20160426044609
This problem occurs on LINUX, but not on Windows. (I do not know on OSX)
Flags: needinfo?(alice0775)
Comment 16•9 years ago
|
||
Hmm, I guess it doesn't happen on OS X then, at least not for me.
It looks like this code is getting rewritten in bug 1233086 (patches almost ready to land it looks like). Before looking into this I would check if those patches fix this. I'll do this eventually if someone doesn't get to it before me (assuming I can reproduce on another machine).
Depends on: 1233086
Updated•9 years ago
|
Flags: needinfo?(howareyou322)
Updated•9 years ago
|
status-firefox49:
--- → affected
Updated•9 years ago
|
Comment 18•9 years ago
|
||
waiting on bug 1233086 for another cycle.
Updated•8 years ago
|
Updated•8 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•