Bug 13366 – REGRESSION (r10935): AOL beta music player doesn't play

• Component: DOM
• Status: RESOLVED FIXED

[Troy Brandt]

If you visit the page above you'll be presented with a list of songs. Click the play link to the right to open a pop-up with the player contained within. 

In Safari Version 2.0.4 (419.3) the song immediately starts playing. In WebKit r20897 the player appears to load but doesn't ever start playing and the controls don't work. It looks like it fails to load up the song for play. 

I'll included a reduction however someone may be able to get it reduced down a bit further.

[Troy Brandt]

Created attachment 14056 [details]
reduction

[David Carson]

According to the Javascript console, there was a TypeError: NULL value on line 1382 of:
http://nsvapp-prod-rr.stream.aol.com:80/283/qt.js

[David Carson]

Created attachment 15077 [details]
The qt script file referenced in Javascript console

[David Carson]

Full details of JS console error:
[21325] http://nsvapp-prod-rr.stream.aol.com:80/fugu?t=uni&tsize=uni&mac=1&tzone=-8&pmmsid=1803379&s1=Fergie&s2=The%20Dutchess&s3=%27Glamorous%27%20feat.%20Ludacris&live=0&angle=0&startpos=0&startpos=0&lpos=0&ppos=0&poffset=0&event=&maxbps=0&skin=grey&width=320&height=77&aspect=4&video=false&showui=1&prefs=1&pid=DL&brand=AOL&player=fls&len=246&country=US&company=AOL&version=1.0&franchise=MUSIC%3A%20Audio%20General&showads=0&locale=US&autoplay=true&mtrack=1&curtainExpand=top&gender=&age=&daypart=&dma=&genre=MUSIC%20-%20Pop&cpid=pmmsid%3A1803379&windowedFullscreen=true&surfacePoint=us.aol.music.newsongs_newsongs&sync=1&restartUrl=mode%3D0%26pmmsid%3D1803379%26referer%3Dhttp%253A//music.aol.com/songs/newsongs line 1382: TypeError: Null value

[David Carson]

using nightly.webkit.org, I went back to the first release, r11976, and the problem still exists there.

[David Carson]

regression was introduced in r10936 when correcting the bug:
http://bugs.webkit.org/show_bug.cgi?id=5449
Added mitz to CC list as he corrected the bug

[David Kilzer (:ddkilzer)]

Code change:  http://trac.webkit.org/projects/webkit/changeset/10935
Layout test: http://trac.webkit.org/projects/webkit/changeset/10936

[David Kilzer (:ddkilzer)]

<rdar://problem/5340098>

[David Kilzer (:ddkilzer)]

7/18/07 3:56 PM Anders Carlsson:
Here's the problem:

The qt.js file has a typo, causing it to write

document.write("<embed"
                + "\" width=" + width + " height=" + height

this will result in a tag with the name <embed" (note the extra character at the end). This causes Safari to not recognize the embed element.

[David Kilzer (:ddkilzer)]

Should we evangelize AOL to fix this issue as well?

[David Kilzer (:ddkilzer)]

Fixed by andersca in r 24429.

[David Kilzer (:ddkilzer)]

(In reply to comment #10)
> Should we evangelize AOL to fix this issue as well?

Sent feedback to AOL via the "Send Us Feedback" link on http://music.aol.com/songs/newsongs.