602425 - Two videos load on GoMobile site

Status: VERIFIED FIXED

(www.mozilla.org :: General, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

Arg; sorry for missing this :-(

If you load http://www-trunk.stage.mozilla.com/en-US/mobile/beta/gomobile/#video using Firefox 3.6.10 and click on the "You are invited" play button, it looks like two videos load; the real one is the 2nd video, and you have to close the first to see it.

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

http://screencast.com/t/XrOQMZssL3

Comment 2

[Stephen Donner [:stephend] Not actively reading bugmail]

21:08:11.667: Network: GET http://videos-cdn.mozilla.net/serv/mobile/meetFFXmobile2-640x360.webm [HTTP/1.1 206 Partial Content
21:08:11.670: Network: GET http://videos-cdn.mozilla.net/serv/mobile/Firefox_Android_invite2.webm [HTTP/1.1 206 Partial Content

Comment 3

[Craig Cook (:craigcook)]

Fixed in r75421.

gomobile.js still had the old video triggered by the same button, but main.js has the new video. Both were triggered by the same click, with the old video loading on top of the new one because that JS file is linked second in the head.

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #3)
> Fixed in r75421.

Thanks! Verified; you, sir, are a godsend.

Comment 5

[Les Orchard [:lorchard]]

This is still broken: It's not new vs old - there *is* a different video that should appear on this page for the "You're invited" video. I'll take a look

Comment 6

[Les Orchard [:lorchard]]

(In reply to comment #5)
> This is still broken: It's not new vs old - there *is* a different video that
> should appear on this page for the "You're invited" video. I'll take a look

(eg. that is, the video from main.js is *not* the video intended to play on this page - the one from gomobile.js is)

Comment 7

[Les Orchard [:lorchard]]

r75443 has the "in action" video on mobile/beta and the "you're invited" video on mobile/beta/gomobile

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #7)
> r75443 has the "in action" video on mobile/beta and the "you're invited" video
> on mobile/beta/gomobile

Thanks, Les -- my mistake.  Verified again on trunk.

Comment 9

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED on prod: http://www.mozilla.com/en-US/mobile/beta/gomobile/#video and http://www.mozilla.com/en-US/mobile/beta/.