Alice0775 White Reporter
	Description • 9 years ago

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

	Aissen
	Comment 1 • 9 years ago

Also reported here: 1125558

	Matthew N. [:MattN]
	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.

	Matthew N. [:MattN]
	Comment 4 • 9 years ago

Oops, I meant 44.

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 6 • 9 years ago

Tracked for 44 since this also impacts push notifications feature to be shipped in FF44.

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 7 • 9 years ago

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.

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 8 • 9 years ago

Josh, Andrew: This bug is about losing the first notification with remote images. Could you help find an owner for this bug? Thanks!

	Matthew N. [:MattN]
	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.

	Andrew Overholt [:overholt]
	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.

	Josh Matthews [:jdm]
	Comment 11 • 9 years ago

Nothing to add to the above.

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 12 • 9 years ago

We are in RC mode, it's too late and this is now a wontfix for Fx44.

	u279076
	Comment 13 • 8 years ago

Milan, who can look at this?

	Timothy Nikkel (:tnikkel)
	Comment 14 • 8 years ago

Does this reproduce anymore? comment 0 worked as expected for me in nightly and release.

	Alice0775 White Reporter
	Comment 15 • 8 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)

	Timothy Nikkel (:tnikkel)
	Comment 16 • 8 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).

	Jim Mathies [:jimm]
	Comment 18 • 8 years ago

waiting on bug 1233086 for another cycle.