Closed Bug 781796 Opened 12 years ago Closed 12 years ago

Can't play cnet videos

Categories

(Tech Evangelism Graveyard :: English US, defect)

defect
Not set
normal

Tracking

(firefox18- affected, firefox19- affected)

RESOLVED FIXED
Tracking Status
firefox18 - affected
firefox19 - affected

People

(Reporter: lcamacho, Unassigned)

References

Details

(Keywords: regression)

User Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/17.0 Firefox/17.0
Build ID: 20120810030512

Steps to reproduce:

Go to this adress http://cnettv.cnet.com/dell-xps-15-hands/9742-1_53-50129172.html and wait for the video to play.


Actual results:

The video never show, in fact the video player never shows.


Expected results:

The video should play.

The firebug console show me this error:

TypeError: N.getNext(...) is null
...mousemove:this.bound.check,mouseup:this.bound.cancel});this.document.addEvent(th...

in http://asset0.cbsistatic.com/cnwk.1d/html/rb/js/tron/cnettv/cnettv.tron.frontdoor.compressed.js line 4
Confirmed in OS X 10.6 nightly. This page works in 16.0a2, so a regression window would be nice.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Hardware: x86 → All
Regression window(m-i)
Good:
http://hg.mozilla.org/integration/mozilla-inbound/rev/aa100b9efcca
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/17.0 Firefox/17.0 ID:20120803104127
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/87e7abe891a9
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/17.0 Firefox/17.0 ID:20120803113827
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=aa100b9efcca&tochange=87e7abe891a9

suspected: Bug 772733
Assignee: nobody → general
Component: Untriaged → JavaScript Engine
Product: Firefox → Core
It does seem to be using a .contains() method on String a few times. I don't know how worked originally. Browser detection gone wrong?
I just test Firefox 14 with the user agent of Nightly and the video works, is not the user agent.
String.contains defines in http://asset3.cbsistatic.com/cnwk.1d/html/rb/js/tron/oreo.moo.rb.combined.js

So , does MooTools conflict with Bug 772733?
It looks like MooTool's contains() is incompatible with ES6's.
Assignee: general → english-us
Component: JavaScript Engine → English US
Product: Core → Tech Evangelism
Version: 17 Branch → unspecified
I should note only Mootools 1.2 is afflicted. Mootools >= 1.3 is fine.
Blocks: 789036
No longer blocks: harmony:stringextras
Well, Cnet changed their website. Now it seems to be fully compatible with Firefox 18. We'll see if it stays that way.
Great!
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Alright. So that was a one time thing, and they are now back to the old layout. This means, of course, it no longer works. This is getting quite annoying.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Blocks: 793781
No longer blocks: 793781
I started tracking firefox-19, as the problem is still there.
(In reply to infinitefirefox from comment #16)
> I started tracking firefox-19, as the problem is still there.

WFM.. latest hourly cset: https://hg.mozilla.org/mozilla-central/rev/ec10630b1a54
win32 m-c trunk on win7 x64
Indeed, they're still using MooTools 1.2 AFAICT.
Is anybody still able to reproduce this? I can't in the latest 10/16 nightly using the URL in comment 0.
Videos do indeed work. However the slideshow is still broken. Since all slideshow related issues seem to be forwarding here, I don't think we can resolve the bug. Although, since the two issues seem to not be related anymore, perhaps we should close this one and open back up the others.
Alright. I believe this issue is resolved. I'm going to reopen my Slideshow issue though.
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Not tracking it for release as the issue no longer happens based on comment 20
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in before you can comment on or make changes to this bug.