189602 - [FIX]Mozilla no longer can play mp3 streams from shoutcast.com (missing *.pls extension)

Status: VERIFIED FIXED

(Core Graveyard :: File Handling, defect, P1)

Description

[afunkebr]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3b) Gecko/20030117
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3b) Gecko/20030117

When clicking on shoutcast.com's "Tune in" link (as the one given above as an
example), mozilla stores the "playlist" file as /tmp/shoutcast-playlist
(WHITHOUT THE .pls extension) and calls the configured helper application
(usually xmms or realplayer), which doesn't recognize the playlist because of
the lacking extension.

It used to work fine in previous mozilla builds (at least it works in build
2002112608), which creates the temporary playlist with a *.pls extension
(/tmp/shoutcast-playlist.pls).

Reproducible: Always

Steps to Reproduce:
1.In mozilla preferences, configure xmms or realplay as the helper app for the
mime type audio/x-scpls (extension .pls)
2.Go to www.shoutcast.com -> listen
3.Click on any of the yellow "Tune in" buttons (sample URL provided above)

Actual Results:  
Xmms (or realplayer) start but cannot recognise the playlist (xmms displays a
"open file dialog" instead, and realplayer complains it doesn't know how to
handle that kind of file).

Expected Results:  
Created the temporary file with the *.pls extension, so that the helper app can
recognise the input file as a playlist (with one or several URLs)

Comment 1

[Boris Zbarsky [:bzbarsky]]

This is a regression from bug 172693.  See item #3 under bug 172693, comment 1.

In this case it seems that .pls is not mapped to that MIME type, hence we do not
get the right extension...

I think I would be in favor of backing out that chunk of the patch until we have
a better solution (not sure what that would be, to be truthful).

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: Proposed fix; just revert that change

Comment 3

[Boris Zbarsky [:bzbarsky]]

fixed.

Comment 4

[Chris Petersen]

Verified in the Linux 2003-01-23-05 trunk build under RH 7.3.