Closed
Bug 121565
Opened 23 years ago
Closed 15 years ago
clicks4u.com - site spawns infinite timers locking up Browser
Categories
(Tech Evangelism Graveyard :: English US, defect, P3)
Tech Evangelism Graveyard
English US
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bc, Unassigned)
References
()
Details
(Whiteboard: aok)
from bug 120475#c14 ------- Additional Comment #14 From Stuart Parmenter 2002-01-23 19:13 ------- The problem is in http://www.clicks4u.com/watermark.js at the bottom, it calls setInterval(), which sucks because it really wants setTimeout, but that isn't what is killing us. the code: if (navigator.appName == "Netscape") { if (navigator.appVersion.indexOf("6") == -1) { wm = document.jsbrand; } } should be checking for nav4 it should be doing (indexOf("5") != -1) for NS6/moz and then using |wm = document.getElementById("jsbrand")| What happens is it fails to set wm, then calls setInterval again. So the first time through, we now have 2 repeating timers... then 4.. 8... 16... Unless JS can detect this and stop it, there isn't much we can do. == Contact Info: support@clicks4u.com
Reporter | ||
Comment 1•23 years ago
|
||
This problem is not Mozilla specific. If you run IE long enough the timers send the CPU to 100%, so there is good motivation for them to fix the script. -> Change Summary to reflect any browser. An example of this problem appears in http://www.jimamidon.com/affiliates-quick.html Contacted late January, follow up in March.
Status: NEW → ASSIGNED
Priority: -- → P3
Summary: clicks4u.com - site spawns infinite timers locking up Mozilla → clicks4u.com - site spawns infinite timers locking up Browser
Target Milestone: --- → Mar
Comment 2•23 years ago
|
||
Note there are other evangelism issues at this site. |document.all|is used without any Moz/N6 codepath. In fact, |document.all| is in functions that get called by error handlers at the site. In Moz/N6, this can cause infinite recursion if the error handler is ever triggered. See http://bugzilla.mozilla.org/show_bug.cgi?id=120475#c2 and the discussion following -
Updated•23 years ago
|
Whiteboard: aok
Comment 3•23 years ago
|
||
Mass reassign of all tech-evangelism us general bugs assigned to bc to doron except bc's P1 bugs. You may search for this mass reassign (it is 305 bugs) by searching for the keyword 'greeneggsandham'
Assignee: bclary → doronr
Status: ASSIGNED → NEW
Target Milestone: Mar → ---
Reporter | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → Mar
Reporter | ||
Comment 4•21 years ago
|
||
not fixed.
Assignee: doron → english-us
Status: ASSIGNED → NEW
QA Contact: zach → english-us
Target Milestone: Mar → ---
Comment 5•15 years ago
|
||
dead domain
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•