Web-bugs: www.deezer.com - site is not usable

[webcompat-bot]

URL: http://www.deezer.com/

Browser / Version: Firefox 57.0.3
Operating System: Windows 7
Tested Another Browser: Yes

Problem type: Site is not usable
Description: The page turns and remains blank when notifications disabled in Firefox
Steps to Reproduce:
1) Disable notifications by switching the dom.webnotifications.enabled flag to false in about:config
2) Visit www.deezer.com
3) The page displays for a split second after which it turns and remains blank

The problem cannot be replicated using Chrome with notifications disabled.

https://bugzilla.mozilla.org/show_bug.cgi?id=1427893

_From webcompat.com with ❤️_

[MDTsai]

@adamopenweb @miketaylr could you help to test this? deezer has georestriction for US only I think.

[softvision-sergiulogigan]

I cannot reproduce this on Windows 10, Nightly 59 or Release 57.
Also tried using a US VPN connection.

[ghost]

@softvision-sergiulogigan Hi, mate. Please ensure you are logged in to the website.

[miketaylr]

This also doesn't reproduce for me, in a new profile (w/ the pref for notifications set to false) on 59.

If the original reporter could test in Firefox Nightly, that would be helpful. Also, with a fresh profile.

[miketaylr]

@softvision-sergiulogigan Hi, mate. Please ensure you are logged in to the website.

Ah, that's what I had wondered. Let me sign up for a trial account.

[ghost]

The bug is replicable on the Nightly build with a fresh profile.

[miketaylr]

Thanks for checking @guoqiyun. Will dig into this in a bit once I've got a trial account working.

[miketaylr]

OK, yeah. Can reproduce. Looking...

[miketaylr]

hello...

ReferenceError: Notification is not defined[Learn More]
app-web.cf3731bfb6a8fb2e6109.js:1:337220

(and then a big ugly ReactJS stack:

OK, so in http://e-cdn-files.deezer.com/cache/js/app-web.cf3731bfb6a8fb2e6109.js, inside some React component, before it mounts they have the following:

t.prototype.componentWillMount = function() {
   "default" === Notification.permission && Notification.requestPermission()
}

If window.Notification is undefined, this will throw. They need to use something like:

if (Notification) {
  "default" === Notification.permission && Notification.requestPermission()
}

(or similar).

[miketaylr]

@adamopenweb would you be able to reach out to Deezer on our internal mailing list? Thanks.

[adamopenweb]

Contacted via mailing list.

[adamopenweb]

Heard back that they are planning to fix this soon.

[ghost]

I would like to add it is not replicable on Firefox Nightly thanks to the new 'Block new requests asking to allow notifications' option in the settings. Switching the flag dom.webnotifications.enabled to false from about:config is not required to block all notifications on Firefox Nightly.

[neovov]

Hi guys,

I'm working at Deezer. This issue should be fixed from now.
Thanks for the report. Feel free to contact us.

Cheers 🍻 .

[adamopenweb]

That's awesome, thank you @neovov!

Closing this report.

[karlcow]

Thanks a lot @neovov (long time no see)