1313322 - Allow mp3 sound files for OS X notifications

Status: RESOLVED FIXED

(Thunderbird :: OS Integration, defect)

Description

[aleth [:aleth]]

In the process of bug 947776, it was noticed that TB doesn't allow selecting mp3 files on this platform, but playing a mp3 file seems to be supported by the underlying code.

Comment 1

[aleth [:aleth]]

Attachment: Allow mp3 sound files for OS X notifications

This seems to work. Unfortunately I couldn't find a list of supported file formats - maybe I was looking in the wrong place?

Comment 2

[Richard Marti (:Paenglab)]

Comment on attachment 8805072 [details] [diff] [review]
Allow mp3 sound files for OS X notifications

I suppose you tested it. rs=me

Comment 3

[Javi Rueda :javirid]

Comment on attachment 8805072 [details] [diff] [review]
Allow mp3 sound files for OS X notifications

Review of attachment 8805072 [details] [diff] [review]:
-----------------------------------------------------------------

From the documentation on Apple, there doesn't seem to be any kind of restriction on the file formats supported by the NSUserNotification API. Right now we are still not using it to customize the sound played along the alert. However, as have been tested with an .mp3 and it worked, there seems to be no problem for the operating system to play .MP3 files when Mozilla toolkit asks it.

Comment 4

[Javi Rueda :javirid]

(In reply to aleth [:aleth] from comment #1)
> Created attachment 8805072 [details] [diff] [review]
> Allow mp3 sound files for OS X notifications
> 
> This seems to work. Unfortunately I couldn't find a list of supported file
> formats - maybe I was looking in the wrong place?

Those file formats would be https://developer.apple.com/library/content/documentation/MusicAudio/Conceptual/CoreAudioOverview/SupportedAudioFormatsMacOSX/SupportedAudioFormatsMacOSX.html, I suppose, as they are the natively supported sound file formats by macOS. I haven't tested them.

Comment 5

[aleth [:aleth]]

https://hg.mozilla.org/comm-central/rev/da656d21e8701d1040ee392c290259fbded49bdc
Bug 1313322 - Allow mp3 sound files for OS X notifications. r=paenglab

Comment 6

[aleth [:aleth]]

(In reply to Javi Rueda from comment #4)
> Those file formats would be
> https://developer.apple.com/library/content/documentation/MusicAudio/
> Conceptual/CoreAudioOverview/SupportedAudioFormatsMacOSX/
> SupportedAudioFormatsMacOSX.html, I suppose, as they are the natively
> supported sound file formats by macOS. I haven't tested them.

Good to know, thanks! I only tested mp3 though.