clicks4u.com - site spawns infinite timers locking up Browser

RESOLVED FIXED

Status

P3
normal
RESOLVED FIXED
17 years ago
3 years ago

People

(Reporter: bc, Unassigned)

Tracking

Details

(Whiteboard: aok, URL)

(Reporter)

Description

17 years ago
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

17 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

17 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

17 years ago
Whiteboard: aok

Comment 3

17 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

17 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → Mar
(Reporter)

Comment 4

15 years ago
not fixed.
Assignee: doron → english-us
Status: ASSIGNED → NEW
QA Contact: zach → english-us
Target Milestone: Mar → ---
dead domain
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in before you can comment on or make changes to this bug.